summaryrefslogtreecommitdiffstats
path: root/courgette
diff options
context:
space:
mode:
authorbrettw <brettw@chromium.org>2015-08-31 15:17:39 -0700
committerCommit bot <commit-bot@chromium.org>2015-08-31 22:18:18 +0000
commitba7a73dd61db639f42cc6283939f8b44cc4ecc6c (patch)
tree8a39315021584d7097ff63d478e9a94c045f9adc /courgette
parentd1ffb22d3c2864efbad907cd690fef7b31b585b9 (diff)
downloadchromium_src-ba7a73dd61db639f42cc6283939f8b44cc4ecc6c.zip
chromium_src-ba7a73dd61db639f42cc6283939f8b44cc4ecc6c.tar.gz
chromium_src-ba7a73dd61db639f42cc6283939f8b44cc4ecc6c.tar.bz2
Annotate GN executables and shared_libraries with sanitizer deps.
These are needed to link in asan/lsan/etc. mode. GYP injects these automatically but GN doesn't have automatic dependency injection. All tests and components (which cover the vast majority of such targets in actual use) are templates which add these dependencies already, so most of the time it never comes up. Only a few of these changes are actually necessary to link all tests run on the asan bot (which is also what engineers will typically build, since almost nobody will build all in asan mode). But to reduce confusion from linker errors in asan mode, I annotated all executables and shared libraries I could find with the sanitizer dependency. I moved a config out of a target in yasm, and sorted a fews deps on other targets. TBR=xhwang@chromium.org (widevine) TBR=achuith@chromium.org (tools/perf, tools/telemetry) Review URL: https://codereview.chromium.org/1318343003 Cr-Commit-Position: refs/heads/master@{#346490}
Diffstat (limited to 'courgette')
-rw-r--r--courgette/BUILD.gn2
1 files changed, 2 insertions, 0 deletions
diff --git a/courgette/BUILD.gn b/courgette/BUILD.gn
index 8826ba2..1ebd2ee 100644
--- a/courgette/BUILD.gn
+++ b/courgette/BUILD.gn
@@ -75,6 +75,7 @@ executable("courgette") {
deps = [
":courgette_lib",
"//base",
+ "//build/config/sanitizers:deps",
]
}
@@ -86,6 +87,7 @@ executable("courgette_minimal_tool") {
deps = [
":courgette_lib",
"//base",
+ "//build/config/sanitizers:deps",
]
}