This is an archive of the discontinued LLVM Phabricator instance.

[sanitizer] [lsan] Add support for AArch64
ClosedPublic

Authored by zatrazz on Oct 12 2015, 1:27 PM.

Details

Summary

This is respin of the previous attempt to enable lsan on aarch64
by fixing the android build issues with missing symbols. This was
due wrong constraint when set CAN_SANITIZE_LEAKS (it should
not be enable for Android).

This patch adds support for leak sanitizer for aarch64. Similar to
MIPS it uses a SizeClassAllocator32 due VMA constraints
(aarch64 currently supports 39 and 42-bit VMA).

Diff Detail

Event Timeline

zatrazz updated this revision to Diff 37157.Oct 12 2015, 1:27 PM
zatrazz retitled this revision from to [sanitizer] [lsan] Add support for AArch64.
zatrazz updated this object.
zatrazz added reviewers: kcc, pcc, rengolin, eugenis, samsonov.
zatrazz added a subscriber: llvm-commits.
eugenis accepted this revision.Oct 13 2015, 11:15 AM
eugenis edited edge metadata.

LGTM

lib/lsan/lsan_common.h
25

Remove all braces on this line (25).

This revision is now accepted and ready to land.Oct 13 2015, 11:15 AM
zatrazz closed this revision.Oct 21 2015, 6:10 AM