set strip_absolute_paths_from_debug_symbols for non goma build too
Previous try failed due to non goma build did not set
strip_absolute_paths_from_debug_symbols.
https://ci.chromium.org/p/chromium/builders/ci/Deterministic%20Linux%20%28dbg%29/6063
I will re-try local build vs remote build comparison after submitting
this CL.
Bug: 953183
Change-Id: I6365101e4934de08a731caabd6db7cdfcedb5d61
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1568359
Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Reviewed-by: Nico Weber <thakis@chromium.org>
Commit-Queue: Takuto Ikuta <tikuta@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#651589}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: b5dfb80b32292f7d8f8423dea388aa4836341f88
diff --git a/compiler/compiler.gni b/compiler/compiler.gni
index f9bb893..3bdd85b 100644
--- a/compiler/compiler.gni
+++ b/compiler/compiler.gni
@@ -194,11 +194,8 @@
# in turn is important for goma compile hit rate.
# Setting this to true may make it harder to debug binaries on Linux, see
# https://chromium.googlesource.com/chromium/src/+/master/docs/linux_debugging.md#Source-level-debug-with-fdebug_compilation_dir
- # So only enable this on linux if use_goma. Elsewhere, there are no
- # drawbacks to using it, so always enable it there.
strip_absolute_paths_from_debug_symbols =
- is_android || is_fuchsia || is_nacl || (is_win && use_lld) ||
- (use_goma && is_linux)
+ is_android || is_fuchsia || is_nacl || (is_win && use_lld) || is_linux
}
# If it wasn't manually set, set to an appropriate default.