[PATCH] D15642: [asan] Use private aliases for global variables (LLVM part).

Maxim Ostapenko via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 2 11:40:51 PST 2016

m.ostapenko added inline comments.

Comment at: lib/Transforms/Instrumentation/AddressSanitizer.cpp:104
@@ -103,2 +103,3 @@
 static const char *const kAsanGenPrefix = "__asan_gen_";
+static const char *const kODRGenPrefix = "__odr_gen_";
 static const char *const kSanCovGenPrefix = "__sancov_gen_";
samsonov wrote:
> Consider naming this to __asan_odr_gen, so that we could know that these new globals are smth. added by ASan.
I'd like to, but I hit on this in Firefox: ASan explicitly exports all **__asan_*** symbols and if we compile our code with e.g -fvisibility=hidden, we can have false positive ODR violation errors. 



More information about the llvm-commits mailing list