This is an archive of the discontinued LLVM Phabricator instance.

[MSSA][NewPM] Handle tests with -print-memoryssa
ClosedPublic

Authored by aeubanks on Aug 5 2020, 11:15 AM.

Details

Summary

-print-memoryssa in legacy PM is print<memoryssa> in NPM.
Pin tests with -print-memoryssa to legacy PM.
Add corresponding tests for NPM where missing.
This fixes "unknown pass name 'print-memoryssa'".

Some tests still fail in Analysis/MemorySSA due to other passes that
haven't been ported.

pr43427.ll and pr43438.ll required adding -aa-pipeline=basic-aa,
-loop-simplify (since it doesn't run on legacy PM by default), and
decrementing some of the MemoryPhi numbers.

Diff Detail

Event Timeline

aeubanks created this revision.Aug 5 2020, 11:15 AM
aeubanks requested review of this revision.Aug 5 2020, 11:15 AM
asbirlea accepted this revision.Aug 5 2020, 11:43 AM
This revision is now accepted and ready to land.Aug 5 2020, 11:43 AM
aeubanks updated this revision to Diff 283421.Aug 5 2020, 3:32 PM

Update some failing tests

aeubanks edited the summary of this revision. (Show Details)Aug 5 2020, 3:32 PM
This revision was landed with ongoing or failed builds.Aug 5 2020, 4:02 PM
This revision was automatically updated to reflect the committed changes.