[PATCH] [asan] Implement delayed activation of AddressSanitizer

Evgeniy Stepanov eugenis at google.com
Thu Jan 16 05:23:20 PST 2014


  Addressed glider's comments in r199380.


================
Comment at: lib/asan/asan_activation.h:21
@@ +20,3 @@
+void AsanActivate();
+} // namespace __asan
+
----------------
Alexander Potapenko wrote:
> 2-spaces before the comment here and in other places.
done

================
Comment at: lib/asan/asan_activation.cc:32
@@ +31,3 @@
+void AsanStartDeactivated() {
+  VReport(1, "Deactivating asan\n");
+  // Save flag values.
----------------
Alexander Potapenko wrote:
> So what about proper naming here? ("ASan" vs. "asan")
done

================
Comment at: lib/asan/asan_malloc_mac.cc:44
@@ -43,3 +43,3 @@
                              vm_size_t start_size, unsigned zone_flags) {
-  if (!asan_inited) __asan_init();
+  if (!asan_inited) AsanInitFromRtl();
   GET_STACK_TRACE_MALLOC;
----------------
Alexander Potapenko wrote:
> Shall we have a macro for this?
done


http://llvm-reviews.chandlerc.com/D2553



More information about the llvm-commits mailing list