[PATCH] D25026: [asan] Move instrumented null-terminated strings to a special section

Kuba Brecka via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 21 09:25:11 PDT 2016

kubabrecka added a comment.


Comment at: projects/compiler-rt/test/asan/TestCases/Darwin/odr-lto.cc:6
-// RUN: %clangxx_asan -DPART=0 -c %s -o %t-1.o -flto
-// RUN: %clangxx_asan -DPART=1 -c %s -o %t-2.o -flto
-// RUN: %clangxx_asan %t-1.o %t-2.o -o %t -flto
-// RUN: not %run %t 2>&1 | FileCheck %s --check-prefix=CHECK-ODR
 // RUN: %clangxx_asan -DPART=0 -c %s -o %t-1.o -flto -mllvm -asan-use-private-alias
 // RUN: %clangxx_asan -DPART=1 -c %s -o %t-2.o -flto -mllvm -asan-use-private-alias
filcab wrote:
> What happened here? There's no error any more? Do we still need to use `-asan-use-private-alias`, etc?
This tests checks that -asan-use-private-alias fixes a known bug when using LTO+ASan.  Eventually, I wanted to turn -asan-use-private-alias on by default to get rid of this annoying bug causing many false positives.  This patch fixes accidentally also this bug for a lot of cases (when strings are involved), but I believe there are other (less common) instances of the LTO+ASan bug that aren’t fixed.  I would still like to keep the odr-lto.cc testcase, because it tests that -asan-use-private-alias doesn’t regress.  If you want me to, I can try to create a new test that would trigger the LTO+ASan bug.


More information about the llvm-commits mailing list