[PATCH] asan: optimization experiments
Kostya Serebryany
kcc at google.com
Tue Mar 10 11:14:59 PDT 2015
You'll also need a test intest/Instrumentation/AddressSanitizer/
================
Comment at: AddressSanitizer.cpp:1077
@@ +1076,3 @@
+// to report the actual access size.
+void AddressSanitizer::instrumentUnusualAddress(Instruction *I, Value *Addr,
+ uint32_t TypeSize, bool IsWrite,
----------------
The name is misleading. The address is not unusual, but size or alignment is.
================
Comment at: lib/asan/asan_report.cc:945
@@ -944,1 +944,3 @@
+ // Optimization experiments mask (see the comment in
+ // lib/Transforms/Instrumentation/AddressSanitizer.cpp:instrumentMop).
----------------
the rt is used with other compilers. I would prefer to have a full comment here
================
Comment at: lib/asan/asan_rtl.cc:171
@@ +170,3 @@
+ } \
+ extern "C" NOINLINE INTERFACE_ATTRIBUTE \
+ void __asan_exp_##type##size(uptr addr, u32 exp) { \
----------------
can you avoid so much duplication here?
http://reviews.llvm.org/D8198
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list