This is an archive of the discontinued LLVM Phabricator instance.

[opt] Add -p alias for -passes
ClosedPublic

Authored by aeubanks on Oct 24 2022, 9:43 AM.

Details

Summary

See [1] for background.

Some people have complained that opt -passes=instcombine is a lot more typing than opt -instcombine.
As a compromise that nobody has objected to in [1], allow opt -p instcombine.

[1] https://discourse.llvm.org/t/rfc-legacy-opt-pass-syntax-with-new-pass-manager/65863

Diff Detail

Event Timeline

aeubanks created this revision.Oct 24 2022, 9:43 AM
aeubanks requested review of this revision.Oct 24 2022, 9:43 AM
Herald added a project: Restricted Project. · View Herald TranscriptOct 24 2022, 9:43 AM
bjope accepted this revision.Nov 1 2022, 1:51 PM

LGTM

(Only concern is that -p used to mean something else. So we are re-introducing a short-option for something new. But I doubt that it will cause much confusion.)

This revision is now accepted and ready to land.Nov 1 2022, 1:51 PM
asbirlea accepted this revision.Nov 1 2022, 3:13 PM
This revision was landed with ongoing or failed builds.Nov 2 2022, 10:47 AM
This revision was automatically updated to reflect the committed changes.