Page MenuHomePhabricator

[lsan] Enable LSan on arm Linux, clang part
ClosedPublic

Authored by m.ostapenko on Apr 6 2017, 6:57 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

m.ostapenko created this revision.Apr 6 2017, 6:57 AM
eugenis added inline comments.Apr 6 2017, 12:57 PM
lib/Driver/ToolChains/Linux.cpp
867 ↗(On Diff #94364)

I think we should also check for thumb. We allow asan on thumb, it would be surprising if lsan was not allowed.

m.ostapenko updated this revision to Diff 94489.Apr 7 2017, 1:03 AM

Also check for thumb.

eugenis edited edge metadata.Apr 7 2017, 12:11 PM

I think it should also include armeb and thumbeb for completeness.

m.ostapenko added a project: Restricted Project.

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 course.

This revision is now accepted and ready to land.Apr 10 2017, 11:38 AM
This revision was automatically updated to reflect the committed changes.