This is an archive of the discontinued LLVM Phabricator instance.

[test] Use -passes=instsimplify instead of -instsimplify in a number of tests. NFC
ClosedPublic

Authored by bjope on Feb 6 2022, 7:25 AM.

Details

Summary

Another step moving away from the deprecated syntax of specifying
pass pipeline in opt.

Diff Detail

Event Timeline

bjope created this revision.Feb 6 2022, 7:25 AM
bjope requested review of this revision.Feb 6 2022, 7:25 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 6 2022, 7:25 AM
Herald added a subscriber: aheejin. · View Herald Transcript
aeubanks accepted this revision.Feb 6 2022, 1:22 PM

I think trivial changes like this one don't need a review and should be good to go

This revision is now accepted and ready to land.Feb 6 2022, 1:22 PM
bjope added a comment.Feb 7 2022, 1:49 AM

I think trivial changes like this one don't need a review and should be good to go

Ok, got it. Thanks for trusting that I don't mess up things.
(Usually I want someone from another organization to review my patches, but as you say, for trivial patches like this one I guess it could be OK to just let someone from my own team to make a quick review.)

This revision was landed with ongoing or failed builds.Feb 7 2022, 5:27 AM
This revision was automatically updated to reflect the committed changes.