This is an archive of the discontinued LLVM Phabricator instance.

[LowerMatrixIntrinsics][NewPM] Fix PreservedAnalyses result
ClosedPublic

Authored by aeubanks on Oct 9 2020, 6:05 PM.

Details

Summary

PreservedCFGCheckerInstrumentation was saying that LowerMatrixIntrinsics
didn't properly preserve CFG even though it claimed to. The legacy pass
says it doesn't. Match the legacy pass's preserved analyses.

Diff Detail

Event Timeline

aeubanks created this revision.Oct 9 2020, 6:05 PM
Herald added a project: Restricted Project. · View Herald TranscriptOct 9 2020, 6:05 PM
aeubanks requested review of this revision.Oct 9 2020, 6:05 PM
thakis accepted this revision.Oct 21 2020, 12:17 PM
thakis added subscribers: fhahn, thakis.

Looks like d1fed7081d80 didn't update this as far as I understand. LG.

This revision is now accepted and ready to land.Oct 21 2020, 12:17 PM

Looks like d1fed7081d80 didn't update this as far as I understand. LG.

Yes, thanks for fixing.