[PATCH] D88569: [DAGCombiner] Call SimplifyDemandedBits to simplify EXTRACT_VECTOR_ELT

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 15 05:54:41 PST 2021


RKSimon added inline comments.


================
Comment at: llvm/test/CodeGen/X86/buildvec-insertvec.ll:783
   %5 = insertelement <4 x i32> undef, i32 %3, i32 undef
   store i32 %4, i32* undef
   ret <4 x i32> %5
----------------
this should simplify to "store i32 undef, store i32* undef" and be removed - can you check why it isn't please?


================
Comment at: llvm/test/CodeGen/X86/vec_setcc.ll:223
+; SSE41-NEXT:    pmovsxwd %xmm1, %xmm0
+; SSE41-NEXT:    movd %xmm1, %eax
+; SSE41-NEXT:    pextrb $4, %xmm0, %edx
----------------
why didn't this simplify?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88569



More information about the llvm-commits mailing list