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

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

Issue 2451973004: Enable use-after-scope check in ASAN configs. (Closed)
Patch Set: exclude mac Created 3 years, 11 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/config/sanitizers/BUILD.gn
diff --git a/build/config/sanitizers/BUILD.gn b/build/config/sanitizers/BUILD.gn
index e7425461d40b6cdf5aab428f0adf66ba934dde13..643d245420ff57c47bfed1de184a2c16dda0c262 100644
--- a/build/config/sanitizers/BUILD.gn
+++ b/build/config/sanitizers/BUILD.gn
@@ -200,6 +200,9 @@ config("default_sanitizer_ldflags") {
ldflags = []
if (is_asan) {
ldflags += [ "-fsanitize=address" ]
+ if (!is_mac) {
achuithb 2017/01/21 01:00:40 Also not chromeos
+ ldflags += [ "-fsanitize-address-use-after-scope" ]
+ }
}
if (is_lsan) {
ldflags += [ "-fsanitize=leak" ]
@@ -309,6 +312,9 @@ config("asan_flags") {
cflags = []
if (is_asan) {
cflags += [ "-fsanitize=address" ]
+ if (!is_mac) {
+ cflags += [ "-fsanitize-address-use-after-scope" ]
+ }
if (!asan_globals) {
cflags += [
"-mllvm",
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698