Page MenuHomePhabricator

[LICM] Enable control flow hoisting by default
ClosedPublic

Authored by john.brawn on Nov 27 2018, 6:30 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

mkazantsev accepted this revision.Nov 27 2018, 5:31 PM

LGTM, but I have a suggestion. How about turning the option on explicitly for these tests? So that the "enable by default" patch contains only 1 line and super-easy to revert.

This revision is now accepted and ready to land.Nov 27 2018, 5:31 PM
This revision was automatically updated to reflect the committed changes.