[PATCH] D133972: [AMDGPU] Fix size of SOPK instructions to 4 bytes

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 15 17:23:30 PDT 2022


arsenm added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/SIInstrInfo.cpp:7486
   // the operands to know the size.
-  if (isFixedSize(MI)) {
+  if (isFixedSize(MI) || isSOPK()) {
     unsigned Size = DescSize;
----------------
Shouldn't special case this here, FixedSize should be set.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D133972



More information about the llvm-commits mailing list