This is an archive of the discontinued LLVM Phabricator instance.

[PassBuilder] Add textual representation for function simplification pipeline
ClosedPublic

Authored by aeubanks on Jun 26 2023, 9:55 AM.

Diff Detail

Event Timeline

aeubanks created this revision.Jun 26 2023, 9:55 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 26 2023, 9:55 AM
aeubanks requested review of this revision.Jun 26 2023, 9:55 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 26 2023, 9:55 AM
nikic added a comment.Jun 26 2023, 1:19 PM

Should we also support specifying the LTO phase?

llvm/lib/Passes/PassBuilder.cpp
1066

This StringSwitch is repeated in a few places.

aeubanks updated this revision to Diff 535454.Jun 28 2023, 10:02 AM

factor out opt level parsing

Should we also support specifying the LTO phase?

we can add that later if somebody needs it

nikic accepted this revision.Jun 29 2023, 12:43 AM

LGTM

This revision is now accepted and ready to land.Jun 29 2023, 12:43 AM
This revision was landed with ongoing or failed builds.Jun 29 2023, 9:39 AM
This revision was automatically updated to reflect the committed changes.