Page MenuHomePhabricator

[NewPassManager] Add tuning option: LoopUnrolling [NFC].

Authored by asbirlea on May 6 2019, 4:12 PM.

Diff Detail


Event Timeline

asbirlea created this revision.May 6 2019, 4:12 PM
Herald added a project: Restricted Project. · View Herald TranscriptMay 6 2019, 4:12 PM

Hmm, how should we test that this works?

I think the test in the clang patch that needs this should do?

chandlerc accepted this revision.May 17 2019, 2:54 PM

Eh, I'd like long term to have a way of testing this field in LLVM, but I think this is fine for now to unblock the Clang changes and other changes.

Maybe in a follow-up either a unittest that works like Clang does or add a commandline flag so it can be tested with opt?

This revision is now accepted and ready to land.May 17 2019, 2:54 PM

Ack, will do!

This revision was automatically updated to reflect the committed changes.