[PATCH] D29586: [lsan] Enable LSan for arm Linux
Maxim Ostapenko via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Feb 16 14:09:04 PST 2017
m.ostapenko added a comment.
In https://reviews.llvm.org/D29586#679101, @kcc wrote:
> In https://reviews.llvm.org/D29586#677564, @m.ostapenko wrote:
>
> > Updating. I've enabled slow unwinder for LSan tests on ARM to make them pass regardless of arm/thumb mode and compiler that was used to build LSan runtime.
>
>
> That's pretty bad.
> Yes, the tests will pass, but the tool will be utterly useless in real life. Slow unwinder is just too slow.
> And setting fast_unwind_on_malloc=false in the test config means we are not testing what the users are going to use.
I agree. ARM ABI is pretty bad -- in thumb mode we can't use fast unwinder because of known reasons, but I really don't know how can we reliably run LSAN tests on Arm without enabling slow unwinder. In any case, I cannot push these changes -- you are the code owner after all, but even with slow unwinder we got pretty fine results on our distributiive. Is there any way how we can enable LSAN for Arm?
Repository:
rL LLVM
https://reviews.llvm.org/D29586
More information about the llvm-commits
mailing list