[PATCH] [asan-assembly-instrumentation] Fixed operations with stack in assembly instrumentation.

Evgeniy Stepanov eugenis at google.com
Tue May 20 06:56:16 PDT 2014


================
Comment at: lib/asan/tests/CMakeLists.txt:63
@@ -62,2 +62,3 @@
   "-fsanitize-blacklist=${ASAN_BLACKLIST_FILE}"
+  -mllvm -asan-instrument-assembly
 )
----------------
Please add a simple test case that's affected by this flag!
Something like AddressSanitizer.BitFieldPositiveTest but with asm store or load instead of C assingment.

================
Comment at: lib/asan/tests/asan_asm_test.cc:249
@@ -214,1 +248,3 @@
+
+  TestFlags();
 }
----------------
Could you make it a separate test case?

http://reviews.llvm.org/D3839






More information about the llvm-commits mailing list