[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