Page MenuHomePhabricator

[NewPM] Only non-trivially loop unswitch at -O3 and for non-optsize functions
ClosedPublic

Authored by aeubanks on Tue, Jan 12, 3:17 PM.

Diff Detail

Event Timeline

aeubanks created this revision.Tue, Jan 12, 3:17 PM
aeubanks requested review of this revision.Tue, Jan 12, 3:17 PM
Herald added a project: Restricted Project. · View Herald TranscriptTue, Jan 12, 3:17 PM
SjoerdMeijer accepted this revision.Wed, Jan 13, 12:46 PM

To avoid code-size bloat, this is indeed the behaviour that we want.
Thanks, LGTM.

This revision is now accepted and ready to land.Wed, Jan 13, 12:46 PM
asbirlea accepted this revision.Wed, Jan 13, 1:11 PM

Thank you for confirming @SjoerdMeijer.

aeubanks updated this revision to Diff 316521.Wed, Jan 13, 2:46 PM

fix ARM unswitch test: looks like minsize functions are becoming smaller, which is good

This revision was landed with ongoing or failed builds.Wed, Jan 13, 2:57 PM
This revision was automatically updated to reflect the committed changes.