[PATCH] D32886: [asan] A clang flag to enable ELF globals-gc

Peter Collingbourne via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon May 8 16:32:48 PDT 2017

pcc accepted this revision.
pcc added a comment.


Unfortunate, but I guess this is in the same category as things like `-fsized-deallocation`.

Comment at: lib/Driver/SanitizerArgs.cpp:573
+    // This is a workaround for a bug in gold and enabled by default for non-ELF
+    // targets.
Maybe include a binutils PR reference so that we can remove this eventually?

Comment at: lib/Driver/SanitizerArgs.cpp:729
+    CmdArgs.push_back(
+        Args.MakeArgString("-fsanitize-address-globals-dead-stripping"));
I think you don't need MakeArgString for string literals.



More information about the cfe-commits mailing list