[PATCH] D60243: [LSan][AArch64] Speed-up leak-sanitizer on AArch64 for 47-bit VMA

Brian Rzycki via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 4 14:06:24 PDT 2019


brzycki added a comment.

I think the changes are minimal and make sense given the `define()` blocks for other arches. I'll defer to @kcc on approval though.



================
Comment at: compiler-rt/lib/lsan/lsan_allocator.cc:117
+#endif
+  p = allocator.Allocate(GetAllocatorCache(), size, alignment);
+
----------------
Nit: indent level of `p = ...` is different than line 133 below.


================
Comment at: compiler-rt/lib/lsan/lsan_allocator.cc:163
+#endif
   allocator.Deallocate(GetAllocatorCache(), p);
 }
----------------
spacing again.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D60243/new/

https://reviews.llvm.org/D60243





More information about the llvm-commits mailing list