[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