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

Unified Diff: build/config/sanitizers/sanitizers.gni

Issue 2307103002: Switch to using_sanitizer for archiving llvm-symbolizer. Also, add libFuzzer and AFL explicitly in … (Closed)
Patch Set: Created 4 years, 3 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
« no previous file with comments | « base/BUILD.gn ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/config/sanitizers/sanitizers.gni
diff --git a/build/config/sanitizers/sanitizers.gni b/build/config/sanitizers/sanitizers.gni
index 132512d2529c50cd90ba692f3265dd8a79fe22ca..a005dd4fdb9f7d8a0032b75d7b01c1c53cd6339e 100644
--- a/build/config/sanitizers/sanitizers.gni
+++ b/build/config/sanitizers/sanitizers.gni
@@ -149,9 +149,9 @@ if (use_afl && sanitizer_coverage_flags == "") {
sanitizer_coverage_flags = "edge,indirect-calls,8bit-counters"
}
-using_sanitizer =
- is_asan || is_lsan || is_tsan || is_msan || is_ubsan || is_ubsan_null ||
- is_ubsan_vptr || is_ubsan_security || use_sanitizer_coverage
+using_sanitizer = is_asan || is_lsan || is_tsan || is_msan || is_ubsan ||
+ is_ubsan_null || is_ubsan_vptr || is_ubsan_security ||
+ use_afl || use_libfuzzer || use_sanitizer_coverage
Nico 2016/09/02 18:25:00 Why this change? afl and libfuzzer don't mean usin
assert(!using_sanitizer || is_clang,
"Sanitizers (is_*san) require setting is_clang = true in 'gn args'")
« no previous file with comments | « base/BUILD.gn ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698