[PATCH] D29586: [lsan] Enable LSan for arm Linux

Yury Gribov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 6 09:05:00 PST 2017


ygribov added inline comments.


================
Comment at: lib/sanitizer_common/sanitizer_linux_libcdep.cc:187
+    || defined(__aarch64__) || defined(__powerpc64__) || defined(__s390__) \
+    || defined(__arm__)) && SANITIZER_LINUX && !SANITIZER_ANDROID
 // sizeof(struct pthread) from glibc.
----------------
Misleading indent.


================
Comment at: test/lsan/TestCases/swapcontext.cc:8
+//
+// UNSUPPORTED: arm
 
----------------
Deserves an explanation.


Repository:
  rL LLVM

https://reviews.llvm.org/D29586





More information about the llvm-commits mailing list