[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