Page MenuHomePhabricator

[NewPM] Fix placement of LoopFlatten
ClosedPublic

Authored by aeubanks on Wed, Jan 13, 9:50 PM.

Details

Summary

https://reviews.llvm.org/D90402 was inconsistent with where it put
LoopFlatten between the two pass managers. It also missed adding it to
the non-O1 function simplification pipeline.

PR48738

Diff Detail

Event Timeline

aeubanks created this revision.Wed, Jan 13, 9:50 PM
aeubanks requested review of this revision.Wed, Jan 13, 9:50 PM
Herald added a project: Restricted Project. · View Herald TranscriptWed, Jan 13, 9:50 PM
SjoerdMeijer accepted this revision.Thu, Jan 14, 1:24 AM

I was going to look at this today, but thanks for fixing! :-)
LGTM

This revision is now accepted and ready to land.Thu, Jan 14, 1:24 AM
This revision was landed with ongoing or failed builds.Thu, Jan 14, 9:50 AM
This revision was automatically updated to reflect the committed changes.