Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(7)

Unified Diff: build/config/sanitizers/BUILD.gn

Issue 1158643002: GN: Build with a custom libc++ when using ASAN, TSAN or MSAN. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@gn-instrumented-libraries-prebuilt
Patch Set: Created 5 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: build/config/sanitizers/BUILD.gn
diff --git a/build/config/sanitizers/BUILD.gn b/build/config/sanitizers/BUILD.gn
index 59821511cc275b515589f75eb13082b064f8dc85..65b5af202ff988748e9b5f10ea2483410469c995 100644
--- a/build/config/sanitizers/BUILD.gn
+++ b/build/config/sanitizers/BUILD.gn
@@ -2,6 +2,8 @@
# Use of this source code is governed by a BSD-style license that can be
# found in the LICENSE file.
+import("//build/config/sanitizers/sanitizers.gni")
+
# Contains the dependencies needed for sanitizers to link into executables and
# shared_libraries. Unconditionally depend upon this target as it is empty if
# |is_asan|, |is_lsan|, |is_tsan| and |is_msan| are false.
@@ -13,6 +15,9 @@ group("deps") {
public_configs = [ ":sanitizer_options_link_helper" ]
deps += [ ":options_sources" ]
}
+ if (use_custom_libcxx) {
earthdok 2015/05/25 15:27:59 Shouldn't this be in a separate GN file?
Dirk Pranke 2015/05/25 22:26:43 Are you suggesting that rather than have deps depe
earthdok 2015/05/26 16:43:49 Basically, yes. I was trying to say that the sanit
+ deps += [ "//buildtools/third_party/libc++:libcxx_proxy" ]
+ }
}
config("sanitizer_options_link_helper") {

Powered by Google App Engine
This is Rietveld 408576698