[PATCH] D12709: [PATCH] [sanitizers] [msan] Enable MSAN for aarch64

Adhemerval Zanella via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 9 05:49:29 PDT 2015


zatrazz added inline comments.

================
Comment at: test/msan/strlen_of_shadow.cc:25
@@ +24,3 @@
+  unsigned long vma = (unsigned long)__builtin_frame_address(0);
+  vma = 64 - __builtin_clzll(vma);
+
----------------
rengolin wrote:
> Can't you re-use the one you have above? Maybe this should even be in compiler-rt?
This is for tests only, so I see the best option to add on 'test/msan/test.h'. I will change it.


http://reviews.llvm.org/D12709





More information about the llvm-commits mailing list