This is an archive of the discontinued LLVM Phabricator instance.

[MachineVerifier] Try harder to verify LiveStacks
ClosedPublic

Authored by foad on Jul 6 2022, 6:49 AM.

Details

Summary

Verify the LiveStacks analysis after a pass that claims to preserve it,
even if there are no further passes (apart from the verifier itself)
that would use the analysis.

Diff Detail

Event Timeline

foad created this revision.Jul 6 2022, 6:49 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 6 2022, 6:49 AM
Herald added a subscriber: hiraditya. · View Herald Transcript
foad requested review of this revision.Jul 6 2022, 6:49 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 6 2022, 6:49 AM
MatzeB accepted this revision.Jul 11 2022, 9:31 AM

Seems sensible. Please keep an eye on buildbots etc. I wouldn't be surprised if this breaks here and there. LGTM

This revision is now accepted and ready to land.Jul 11 2022, 9:31 AM
This revision was landed with ongoing or failed builds.Jul 12 2022, 1:57 AM
This revision was automatically updated to reflect the committed changes.