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

Joe Nash via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 15 14:10:05 PDT 2022


Joe_Nash added a comment.

I believe you are correct about SOPK being fixed size. This looks like it could be an NFC change, because the code below would detect if there was a literal used. If you have any reason to think its not NFC, please add a test. That said, I think a better way to do the change is set FixedSize = 1 on SOPP_Pseudo.


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