[PATCH] D137274: AMDGPU/GlobalISel: Fix combine crash because LI is not set in prelegalizer

Petar Avramovic via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 2 11:12:29 PDT 2022


Petar.Avramovic added a comment.

To my understanding prelegalizer now also requires legalizer info?



================
Comment at: llvm/test/CodeGen/AMDGPU/cttz_zero_undef.ll:743
 ; GFX9-GISEL-NEXT:    v_and_b32_e32 v2, 0xffff, v2
-; GFX9-GISEL-NEXT:    v_cmp_ne_u32_e32 vcc, 0, v1
+; GFX9-GISEL-NEXT:    v_cmp_ne_u32_sdwa vcc, v1, v0 src0_sel:WORD_0 src1_sel:DWORD
 ; GFX9-GISEL-NEXT:    v_cndmask_b32_e32 v1, 32, v2, vcc
----------------
Change is caused by zextload being combined pre-legalizer


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

https://reviews.llvm.org/D137274



More information about the llvm-commits mailing list