This is an archive of the discontinued LLVM Phabricator instance.

[test][NewPM] Pin various tests to the legacy PM
ClosedPublic

Authored by aeubanks on Feb 2 2021, 2:50 PM.

Details

Summary

tut-simplify-cfg hasn't been ported to the new PM.
llvm-lto2's -enable-new-pm defaults to the CMake flag, so the legacy PM extension test needs to be pinned.

Diff Detail

Event Timeline

aeubanks requested review of this revision.Feb 2 2021, 2:50 PM
aeubanks created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptFeb 2 2021, 2:50 PM

should be the last set of tests

MaskRay accepted this revision.Feb 2 2021, 2:54 PM

Thanks!

This revision is now accepted and ready to land.Feb 2 2021, 2:54 PM
ychen accepted this revision.Feb 2 2021, 2:54 PM

LGTM

This revision was landed with ongoing or failed builds.Feb 2 2021, 3:07 PM
This revision was automatically updated to reflect the committed changes.
fhahn added a subscriber: fhahn.Feb 3 2021, 12:27 AM

Thanks for pinning these to the old pass manager. Is there a way to add a new PM version of the pass & link conditionally if examples are enabled?

Thanks for pinning these to the old pass manager. Is there a way to add a new PM version of the pass & link conditionally if examples are enabled?

llvm/examples/Bye/Bye.cpp should do just that.