This revision was automatically updated to reflect the committed changes.
Closed by commit rL299921: [lsan] Enable LSan on arm Linux, clang part
(authored by chefmax).
Changed prior to commit:
https://reviews.llvm.org/D31760?vs=94708&id=94787#toc
Repository:
rL LLVM
https://reviews.llvm.org
eugenis accepted this revision.
eugenis added a comment.
This revision is now accepted and ready to land.
LGTM
Repository:
rL LLVM
https://reviews.llvm.org/D31760
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-b
m.ostapenko updated this revision to Diff 94708.
m.ostapenko added a project: Sanitizers.
m.ostapenko added a comment.
Add armeb and thumbeb for completeness.
Just curious, does anyone use sanitizers in these targets? I see no public
buildbots for armeb and thumbeb. Anyway, not a big deal of cour
eugenis added a comment.
I think it should also include armeb and thumbeb for completeness.
Repository:
rL LLVM
https://reviews.llvm.org/D31760
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo
m.ostapenko updated this revision to Diff 94489.
m.ostapenko added a comment.
Also check for thumb.
Repository:
rL LLVM
https://reviews.llvm.org/D31760
Files:
lib/Driver/ToolChains/Linux.cpp
test/Driver/fsanitize.c
Index: test/Driver/fsanitize.c
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::getSupportedSanitize
m.ostapenko created this revision.
Herald added subscribers: rengolin, aemerson.
This is a compiler part of https://reviews.llvm.org/D29586. Enable LSan on arm
Linux.
Repository:
rL LLVM
https://reviews.llvm.org/D31760
Files:
lib/Driver/ToolChains/Linux.cpp
test/Driver/fsanitize.c
Ind
