[PATCH] D139546: [AMDGPU][GlobalISel] Fix legalizing image intrinsics for new types

Mirko Brkusanin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 7 08:49:16 PST 2022


mbrkusanin created this revision.
mbrkusanin added reviewers: foad, sebastian-ne, matejam, arsenm.
mbrkusanin added a project: LLVM.
Herald added subscribers: kosarev, wenlei, kerbowa, hiraditya, tpr, dstuttard, yaxunl, jvesely, kzhuravl.
Herald added a project: All.
mbrkusanin requested review of this revision.
Herald added a subscriber: wdng.

We no longer need to increase vector size to 16 for intrinsics that use more
than 8 vgprs for addr. There is no image intrinsic that needs more than 12
so all currently existing cases will be covered. Using incorrect size was
causing an error in instruction selection because instructions were updated
to require new types.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D139546

Files:
  llvm/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp
  llvm/test/CodeGen/AMDGPU/GlobalISel/legalize-llvm.amdgcn.image.sample.d.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D139546.480931.patch
Type: text/x-patch
Size: 28711 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20221207/16142675/attachment.bin>


More information about the llvm-commits mailing list