Page MenuHomePhabricator

[LoongArch] Override TargetLowering::isOffsetFoldingLegal()
ClosedPublic

Authored by wangleiat on Aug 9 2022, 6:06 AM.

Details

Diff Detail

Event Timeline

wangleiat created this revision.Aug 9 2022, 6:06 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 9 2022, 6:06 AM
wangleiat requested review of this revision.Aug 9 2022, 6:06 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 9 2022, 6:06 AM
xen0n accepted this revision.Aug 9 2022, 6:10 AM

RISCV and CSKY both do the same. This one was easy to review. Thanks for the quick fix!

This revision is now accepted and ready to land.Aug 9 2022, 6:10 AM
xen0n added a comment.Aug 9 2022, 6:11 AM

RISCV and CSKY both do the same. This one was easy to review. Thanks for the quick fix!

Or rather, both contain the same comments ;-)

(Looking closer, CSKY doesn't have isOffsetFoldingLegal, but the core principle stays the same.)

This revision was landed with ongoing or failed builds.Aug 10 2022, 8:29 PM
This revision was automatically updated to reflect the committed changes.