[PATCH] D137274: AMDGPU/GlobalISel: Fix combine crash because LI is not set in prelegalizer
Matt Arsenault via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Nov 2 12:33:49 PDT 2022
arsenm accepted this revision.
arsenm added inline comments.
This revision is now accepted and ready to land.
================
Comment at: llvm/test/CodeGen/AMDGPU/cttz_zero_undef.ll:1619
; GFX9-GISEL-NEXT: v_and_b32_e32 v2, 0xffff, v2
-; GFX9-GISEL-NEXT: v_cmp_eq_u32_e32 vcc, 0, v1
+; GFX9-GISEL-NEXT: v_cmp_eq_u32_sdwa vcc, v1, v0 src0_sel:WORD_0 src1_sel:DWORD
; GFX9-GISEL-NEXT: v_cndmask_b32_e32 v1, v2, v3, vcc
----------------
This is a regression without getting rid of one of these ands
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D137274/new/
https://reviews.llvm.org/D137274
More information about the llvm-commits
mailing list