[PATCH] D15644: [asan] Use private aliases for global variables (compiler-rt part).

Maxim Ostapenko via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 18 11:32:33 PST 2015


m.ostapenko added inline comments.

================
Comment at: test/asan/TestCases/Linux/local_alias.cc:15
@@ +14,3 @@
+long foo(long *);
+long h = 12;
+long i = 13;
----------------
kcc wrote:
> isn't this a real ODR which we now will fail to detect? 
Yes, we can't detect ODR violation here, because libfoo.so is sanitized and libbar.so is not. But how could we do it with existing approach? And anyway, this particular testcase fails with CHECK on x86 and with false positive global buffer overflow on x86_64. I think it's better to miss ODR violation than just miserably die :). 


Repository:
  rL LLVM

http://reviews.llvm.org/D15644





More information about the llvm-commits mailing list