Page MenuHomePhabricator

[NewPM] Don't mark AA analyses as preserved
ClosedPublic

Authored by aeubanks on May 6 2021, 4:33 PM.

Details

Summary

Currently all AA analyses marked as preserved are stateless, not taking
into account their dependent analyses. So there's no need to mark them
as preserved, they won't be invalidated unless their analyses are.

SCEVAAResults was the one exception to this, it was treated like a
typical analysis result. Make it like the others and don't invalidate
unless SCEV is invalidated.

Diff Detail

Event Timeline

aeubanks created this revision.May 6 2021, 4:33 PM
aeubanks requested review of this revision.May 6 2021, 4:33 PM
Herald added a project: Restricted Project. · View Herald TranscriptMay 6 2021, 4:33 PM
aeubanks updated this revision to Diff 343533.May 6 2021, 4:35 PM

a couple more missed places

asbirlea accepted this revision.May 18 2021, 1:29 PM

LGTM.

This revision is now accepted and ready to land.May 18 2021, 1:29 PM
This revision was automatically updated to reflect the committed changes.