[PATCH] D13108: [compiler-rt] [lsan] Add support for AArch64

Adhemerval Zanella via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 24 15:46:11 PDT 2015


zatrazz added a comment.

In http://reviews.llvm.org/D13108#252071, @samsonov wrote:

> Does the LSan test suite pass after this change?


Yes, I see no failures.

In http://reviews.llvm.org/D13108#252388, @rengolin wrote:

> What about the CMake changes?


It does add arm64 as a supported one.


================
Comment at: lib/lsan/lsan_common.h:26
@@ -25,2 +25,3 @@
 #if SANITIZER_LINUX && (defined(__x86_64__) || defined(__mips64)) \
+    || defined(__aarch64__) \
     && (SANITIZER_WORDSIZE == 64)
----------------
samsonov wrote:
> Nope, the brackets are wrong here.
Right, I will fix it.


http://reviews.llvm.org/D13108





More information about the llvm-commits mailing list