[PATCH] D148127: [AMDGPU] Revert "Transform illegal intrinsics to V_ILLEGAL"

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 13 01:17:00 PDT 2023


foad added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/SIInstructions.td:3640-3656
-
-def V_ILLEGAL_gfx6_gfx7_gfx8_gfx9 : Enc32, InstSI<(outs), (ins), "v_illegal"> {
-  let Inst{31-0} = 0xFFFFFFFF;
-  let FixedSize = 1;
-  let Size = 4;
-  let Uses = [EXEC];
-  let hasSideEffects = 1;
----------------
arsenm wrote:
> We should still recognize the encoded instructions (at least of gfx10 where it's defined)
They should have been added in a separate patch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D148127



More information about the llvm-commits mailing list