[PATCH] D15642: [asan] Use private aliases for global variables (LLVM part).
Alexey Samsonov via llvm-commits
llvm-commits at lists.llvm.org
Tue Feb 2 14:23:56 PST 2016
samsonov 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_";
----------------
m.ostapenko wrote:
> 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.
Oh, I see... `__odr_asan_gen_` ? :)
Repository:
rL LLVM
http://reviews.llvm.org/D15642
More information about the llvm-commits
mailing list