[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