This is an archive of the discontinued LLVM Phabricator instance.

[PowerPC][GISel] support 32 bit load/store
ClosedPublic

Authored by shchenz on Oct 9 2022, 1:55 AM.

Details

Reviewers
Kai
arsenm
amyk
nemanjai
Group Reviewers
Restricted Project
Commits
rGb41d22db18ec: [PowerPC][GISel] support 32 bit load/store
Summary

This is a follow up of D134792. This patch supports 32 bit load/store.

Diff Detail

Event Timeline

shchenz created this revision.Oct 9 2022, 1:55 AM
Herald added a project: Restricted Project. · View Herald TranscriptOct 9 2022, 1:55 AM
shchenz requested review of this revision.Oct 9 2022, 1:55 AM
Herald added a project: Restricted Project. · View Herald TranscriptOct 9 2022, 1:55 AM
arsenm accepted this revision.Nov 16 2022, 3:19 PM
This revision is now accepted and ready to land.Nov 16 2022, 3:19 PM
shchenz updated this revision to Diff 478817.Nov 29 2022, 11:26 PM

Make load/store selection be parent of constantFP selection patch D133340

amyk added inline comments.Nov 30 2022, 7:01 AM
llvm/lib/Target/PowerPC/GISel/PPCLegalizerInfo.cpp
53–54

We should update this comment (since as of this patch, we support both 32-bit and 64-bit).

shchenz added inline comments.Nov 30 2022, 4:59 PM
llvm/lib/Target/PowerPC/GISel/PPCLegalizerInfo.cpp
53–54

good catch, will update.

shchenz updated this revision to Diff 479120.Nov 30 2022, 5:23 PM

fix inaccurate comments

shchenz updated this revision to Diff 480360.Dec 6 2022, 12:34 AM

set right legalizer info

This revision was landed with ongoing or failed builds.Dec 12 2022, 4:53 AM
This revision was automatically updated to reflect the committed changes.