[PATCH] D135609: [AMDGPU] Fix True16 patterns for cmp on GFX11

Tue Oct 11 07:27:50 PDT 2022

Leonc added inline comments.

================
Comment at: llvm/test/CodeGen/AMDGPU/v_cmp_gfx11.ll:7
+; CHECK:       ; %bb.0: ; %entry
+; CHECK-NEXT:    v_cmp_eq_u16_e64 s0, 0, 0
+; CHECK-NEXT:    v_mov_b32_e32 v1, 0
----------------
Joe_Nash wrote:
> Leonc wrote:
> > Shouldn't this be `v_cmp_eq_u16_t16_e64` on gfx11?
> The assembly mnemonics do not include references to true16, only the MachineInstr/MCInst names do. I checked that the True16 version of the instruction was used via print-after-all before asm emission in this case.
Excellent. Thanks 👍

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D135609