This is an archive of the discontinued LLVM Phabricator instance.

GlobalISel: Lift store value widening restriction
ClosedPublic

Authored by arsenm on Aug 24 2020, 8:17 AM.

Details

Summary

This doesn't change the memory size and doesn't need to worry about
non-power-of-2 sizes.

Diff Detail

Event Timeline

arsenm created this revision.Aug 24 2020, 8:17 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 24 2020, 8:17 AM
arsenm requested review of this revision.Aug 24 2020, 8:17 AM
paquette added inline comments.Aug 27 2020, 11:00 AM
llvm/test/CodeGen/AMDGPU/GlobalISel/legalize-store-global.mir
459

Why are there commented-out tests here?

arsenm added inline comments.Aug 27 2020, 11:04 AM
llvm/test/CodeGen/AMDGPU/GlobalISel/legalize-store-global.mir
459

These assert for other reasons that need to be fixed

aemerson accepted this revision.Sep 16 2020, 10:43 AM
This revision is now accepted and ready to land.Sep 16 2020, 10:43 AM