[PATCH] D157624: [AMDGPU] Treat KIMM32 and KIMM16 operand types as noninlinable

Joe Nash via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 11 08:35:17 PDT 2023


Joe_Nash added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/SIInstrInfo.cpp:3909
   case AMDGPU::OPERAND_KIMM16:
+    return false;
   case AMDGPU::OPERAND_INPUT_MODS:
----------------
mbrkusanin wrote:
> arsenm wrote:
> > I think this part is unnecessary but the rest looks fine
> Not for the issue of inst size which is fixed with FixedSize, but for a downstream test (see email).
This is a fix for the test noted on here https://reviews.llvm.org/D157103, and setting it to false is necessary for that.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D157624/new/

https://reviews.llvm.org/D157624



More information about the llvm-commits mailing list