This is an archive of the discontinued LLVM Phabricator instance.

Fix CodeGen/pgo-sample-thinlto-summary.c with old PM
ClosedPublic

Authored by thopre on Sep 17 2021, 2:25 AM.

Details

Summary

Re-add -fexperimental-new-pass-manager to
Clang::CodeGen/pgo-sample-thinlto-summary.c for the test to work on
builds that still default to the old pass manager.

Diff Detail

Event Timeline

thopre created this revision.Sep 17 2021, 2:25 AM
thopre requested review of this revision.Sep 17 2021, 2:25 AM
Herald added a project: Restricted Project. · View Herald TranscriptSep 17 2021, 2:25 AM
tejohnson accepted this revision.Sep 17 2021, 7:18 AM

lgtm. And I just was informed in a recent review of one of my patches that we are now removing old PM tests, so if you want you could also go ahead and remove the first 2 lines here.

This revision is now accepted and ready to land.Sep 17 2021, 7:18 AM
thopre updated this revision to Diff 373214.Sep 17 2021, 7:20 AM

Remove old PM tests

This revision was landed with ongoing or failed builds.Sep 17 2021, 7:21 AM
This revision was automatically updated to reflect the committed changes.