[compiler-rt] r230317 - Fix alloca_instruments_all_paddings.cc test to work under higher -O levels (compiler-rt part)

Kuba Brecka kuba.brecka at gmail.com
Tue Feb 24 01:47:33 PST 2015


Author: kuba.brecka
Date: Tue Feb 24 03:47:33 2015
New Revision: 230317

URL: http://llvm.org/viewvc/llvm-project?rev=230317&view=rev
Log:
Fix alloca_instruments_all_paddings.cc test to work under higher -O levels (compiler-rt part)

When AddressSanitizer only a single dynamic alloca and no static allocas, due to an early exit from FunctionStackPoisoner::poisonStack we forget to unpoison the dynamic alloca.  This patch fixes that.

Reviewed at http://reviews.llvm.org/D7810


Modified:
    compiler-rt/trunk/test/asan/TestCases/alloca_instruments_all_paddings.cc

Modified: compiler-rt/trunk/test/asan/TestCases/alloca_instruments_all_paddings.cc
URL: http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/test/asan/TestCases/alloca_instruments_all_paddings.cc?rev=230317&r1=230316&r2=230317&view=diff
==============================================================================
--- compiler-rt/trunk/test/asan/TestCases/alloca_instruments_all_paddings.cc (original)
+++ compiler-rt/trunk/test/asan/TestCases/alloca_instruments_all_paddings.cc Tue Feb 24 03:47:33 2015
@@ -1,4 +1,5 @@
 // RUN: %clangxx_asan -O0 -mllvm -asan-instrument-allocas %s -o %t
+// RUN: %clangxx_asan -O3 -mllvm -asan-instrument-allocas %s -o %t
 // RUN: %run %t 2>&1
 //
 





More information about the llvm-commits mailing list