This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] Use SelectionDAG::isBaseWithConstantOffset in scalar load/store address matching.
ClosedPublic

Authored by craig.topper on Jun 3 2022, 9:08 AM.

Details

Summary

Test changes are because isBaseWithConstantOffset uses computeKnownBits
and that is able to see that an earlier AND instruction guaranteed
alignment so that we can treat an OR as an ADD.

Diff Detail

Event Timeline

craig.topper created this revision.Jun 3 2022, 9:08 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 3 2022, 9:08 AM
craig.topper requested review of this revision.Jun 3 2022, 9:08 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 3 2022, 9:08 AM
reames accepted this revision.Jun 3 2022, 9:48 AM

LGTM

This revision is now accepted and ready to land.Jun 3 2022, 9:48 AM
This revision was landed with ongoing or failed builds.Jun 3 2022, 11:02 AM
This revision was automatically updated to reflect the committed changes.