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

Brian Rzycki via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 15 08:43:50 PDT 2019


brzycki added a comment.

Hi @kcc. @sebpop and I decided on this approach because we were unable to create a base `allocator` class. We ran into pointer coercion issues for 32-bit or 64-bit pointers returned from some of the methods. We either had to mask this with `(void *)` casts at all the callsites. Even if we did this every routine would require a runtime check for 32 vs 64 on aarch64. This latest patch reduces the number of `ifdefs` and is still a compile-time check for every non-aarch64 platform.


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

https://reviews.llvm.org/D60243





More information about the llvm-commits mailing list