[PATCH] D31760: [lsan] Enable LSan on arm Linux, clang part

Evgeniy Stepanov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Apr 6 12:57:53 PDT 2017


eugenis added inline comments.


================
Comment at: lib/Driver/ToolChains/Linux.cpp:867
                          getTriple().getArch() == llvm::Triple::aarch64_be;
+  const bool IsArmArch = getTriple().getArch() == llvm::Triple::arm;
   SanitizerMask Res = ToolChain::getSupportedSanitizers();
----------------
I think we should also check for thumb. We allow asan on thumb, it would be surprising if lsan was not allowed.


Repository:
  rL LLVM

https://reviews.llvm.org/D31760





More information about the cfe-commits mailing list