Page MenuHomePhabricator

[RISCV] Enable MachineOutliner by default under -Oz for RISCV
ClosedPublic

Authored by pcwang-thead on Mar 22 2022, 1:19 AM.

Details

Summary

Enable default outlining when the function has the minsize attribute.

addr-label.ll crashed after enabling this, so a barrier is added before
instruction selection as a workaround.

Diff Detail

Event Timeline

pcwang-thead created this revision.Mar 22 2022, 1:19 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 22 2022, 1:19 AM
pcwang-thead requested review of this revision.Mar 22 2022, 1:19 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 22 2022, 1:19 AM

It's probably a good idea to separate the fixes from the change to the default enable.

Separate patches.

pcwang-thead edited the summary of this revision. (Show Details)Mar 28 2022, 10:16 PM

It's probably a good idea to separate the fixes from the change to the default enable.

I have separated this patch into three patches:

Ping. Is there any problem I should take care of?

luismarques accepted this revision.May 3 2022, 4:07 AM

LGTM. Thanks.

This revision is now accepted and ready to land.May 3 2022, 4:07 AM
This revision was landed with ongoing or failed builds.May 6 2022, 2:38 AM
This revision was automatically updated to reflect the committed changes.