This is an archive of the discontinued LLVM Phabricator instance.

[AMDGPU] Always lower SHUFFLE_VECTOR
ClosedPublic

Authored by Pierre-vh on Sep 30 2022, 7:56 AM.

Details

Summary

Make it illegal, remove InstructionSelector logic for it

Diff Detail

Event Timeline

Pierre-vh created this revision.Sep 30 2022, 7:56 AM
Herald added a project: Restricted Project. · View Herald TranscriptSep 30 2022, 7:56 AM
Pierre-vh requested review of this revision.Sep 30 2022, 7:56 AM
Herald added a project: Restricted Project. · View Herald TranscriptSep 30 2022, 7:56 AM
arsenm accepted this revision.Sep 30 2022, 8:02 AM
arsenm added inline comments.
llvm/lib/Target/AMDGPU/AMDGPUInstructionSelector.cpp
3292–3293

Should just delete this

llvm/lib/Target/AMDGPU/SIInstructions.td
1989–1991 ↗(On Diff #464281)

Overlap with D134463

This revision is now accepted and ready to land.Sep 30 2022, 8:02 AM
Pierre-vh edited the summary of this revision. (Show Details)Oct 4 2022, 12:03 AM
Pierre-vh updated this revision to Diff 464922.Oct 4 2022, 2:19 AM
Pierre-vh marked an inline comment as done.
Pierre-vh edited the summary of this revision. (Show Details)

Rebase on trunk

This revision was automatically updated to reflect the committed changes.
llvm/test/CodeGen/AMDGPU/GlobalISel/legalize-shuffle-vector.s16.mir