[PATCH] Annotate BumpPtrAllocator for MemorySanitizer

Kostya Serebryany kcc at google.com
Thu Jan 31 02:01:24 PST 2013


  LGTM
  I guess we still need someone else's review


================
Comment at: lib/Support/Allocator.cpp:107
@@ +106,3 @@
+    // Update the allocation point of this memory block in MemorySanitizer.
+    // Without this, MemorySanitizer reports for values originating from it will
+    // point to the allocation point of the entire slab.
----------------
to avoid ambiguity I'd rephrase: 

Without this, MemorySanitizer messages for values originated from here will ...


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



More information about the llvm-commits mailing list