This is an archive of the discontinued LLVM Phabricator instance.

[ASAN] Removed special case controlling allocator constants for __aarch64__.
ClosedPublic

Authored by kstoimenov on Nov 2 2022, 9:15 AM.

Details

Summary

This patch should land before D137136 to make sure that the leak sanitizer allocator works correctly. This patch is NFC without D137136.

Diff Detail

Event Timeline

kstoimenov created this revision.Nov 2 2022, 9:15 AM
Herald added a project: Restricted Project. · View Herald TranscriptNov 2 2022, 9:15 AM
kstoimenov requested review of this revision.Nov 2 2022, 9:15 AM
Herald added a project: Restricted Project. · View Herald TranscriptNov 2 2022, 9:15 AM
Herald added a subscriber: Restricted Project. · View Herald Transcript
kstoimenov edited the summary of this revision. (Show Details)Nov 2 2022, 9:21 AM
kstoimenov added a reviewer: vitalybuka.

Please include that this patch is NFC as without D137136 SANITIZER_CAN_USE_ALLOCATOR64 is 0

vitalybuka accepted this revision.Nov 2 2022, 10:17 AM

into description and git commit
note that editing description here does not update the stuff which will be in git commit

This revision is now accepted and ready to land.Nov 2 2022, 10:17 AM
kstoimenov retitled this revision from [ASAN] Removed special case controlling allocator constants for __aarch64__. to [ASAN] Removed special case controlling allocator constants for __aarch64__. This patch is NFC without D137136..

Updated description.

kstoimenov retitled this revision from [ASAN] Removed special case controlling allocator constants for __aarch64__. This patch is NFC without D137136. to [ASAN] Removed special case controlling allocator constants for __aarch64__..
kstoimenov edited the summary of this revision. (Show Details)

Moved NFC comment to the summary.

This revision was landed with ongoing or failed builds.Nov 2 2022, 10:28 AM
This revision was automatically updated to reflect the committed changes.