[PATCH] D102476: [VectorCombine] Use constant range info for index scalarization legality.

Tue Aug 3 09:02:39 PDT 2021

fhahn added inline comments.

================
Comment at: llvm/test/Transforms/VectorCombine/load-insert-store.ll:195
+; CHECK-NEXT:    [[TMP0:%.*]] = getelementptr inbounds <16 x i8>, <16 x i8>* [[Q:%.*]], i32 0, i32 [[IDX_CLAMPED]]
+; CHECK-NEXT:    store i8 [[S:%.*]], i8* [[TMP0]], align 1
 ; CHECK-NEXT:    ret void
----------------
efriedma wrote:
> efriedma wrote:
> > Alive doesn't like this: https://web.ist.utl.pt/nuno.lopes/alive2/index.php?hash=50de39e1c7237e13&test=Transforms%2FVectorCombine%2Fload-insert-store.ll
> Filed as https://bugs.llvm.org/show_bug.cgi?id=50949
Apologies for the delay! I put up a patch limiting the transform to cases where the index is known non-poison: D107364

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102476