This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU/GlobalISel: Fix RegBankSelect for G_INSERT_VECTOR_ELT
ClosedPublic

Authored by arsenm on Jan 3 2020, 5:23 PM.

Details

Summary

The result and source vector are going to be tied, so these need to be
the same bank.

Diff Detail

Event Timeline

arsenm created this revision.Jan 3 2020, 5:23 PM
Herald added a project: Restricted Project. · View Herald TranscriptJan 3 2020, 5:23 PM
arsenm updated this revision to Diff 236161.Jan 3 2020, 7:23 PM

Fix not splitting SGPR 64-bit insert element into VGPR vector

kerbowa accepted this revision.Jan 7 2020, 1:28 PM

LGTM

This revision is now accepted and ready to land.Jan 7 2020, 1:28 PM
kerbowa added inline comments.Jan 7 2020, 1:34 PM
llvm/lib/Target/AMDGPU/AMDGPUGenRegisterBankInfo.def
223

As a general note it's not obvious what are the mappings for SGPROnly64BreakDown/ValMappingsSGPR64OnlyVGPR32.

arsenm closed this revision.Jan 22 2020, 7:58 AM
arsenm marked 2 inline comments as done.

df8f381e4601ccb0a8250b7c24e27f00856c8ce4

llvm/lib/Target/AMDGPU/AMDGPUGenRegisterBankInfo.def
223

I added some comments to try to fix this. This will need to be cleaned up somehow when this is autogenerated