This is an archive of the discontinued LLVM Phabricator instance.

Use store operation to poison allocas for lifetime analysis.
ClosedPublic

Authored by vitalybuka on Aug 26 2016, 4:26 PM.

Details

Summary

Calling asan_poison_stack_memory and asan_unpoison_stack_memory for small
variables is too expensive.

Code is disabled by default and can be enabled by -asan-experimental-poisoning.

PR27453

Diff Detail

Repository
rL LLVM

Event Timeline

vitalybuka updated this revision to Diff 69448.Aug 26 2016, 4:26 PM
vitalybuka retitled this revision from to Use store operation to poison allocas for lifetime analysis..
vitalybuka updated this object.
vitalybuka added a reviewer: eugenis.
vitalybuka added a subscriber: llvm-commits.

Simplified by using arrays

removed debug output

rename function

Shorter names

eugenis accepted this revision.Aug 29 2016, 10:23 AM
eugenis edited edge metadata.

LGTM but fix the ShadowMaskping thing

lib/Transforms/Instrumentation/AddressSanitizer.cpp
368 ↗(On Diff #69478)

:))

This revision is now accepted and ready to land.Aug 29 2016, 10:23 AM
vitalybuka edited edge metadata.

undo accidental renaming

This revision was automatically updated to reflect the committed changes.