[PATCH] D107364: [VectorCombine] Limit scalarization known non-poison indices.

[Eli Friedman via Phabricator via llvm-commits]

efriedma added a comment.

In D107364#2923374 <https://reviews.llvm.org/D107364#2923374>, @fhahn wrote:

> Not sure if there's potential for a convenient API that transforms a constant range which may include poison to one without?

Not sure how useful that would be in general, but maybe.  I mean, "match the and and urem patterns manually" is basically such an API.



================
Comment at: llvm/test/Transforms/VectorCombine/load-insert-store.ll:329
+; CHECK-NEXT:    [[VECINS:%.*]] = insertelement <16 x i8> [[TMP0]], i8 [[S:%.*]], i32 [[IDX_CLAMPED]]
+; CHECK-NEXT:    store <16 x i8> [[VECINS]], <16 x i8>* [[Q]], align 16
 ; CHECK-NEXT:    ret void
----------------
Please make sure we have a corresponding testcase with the appropriate freeze.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D107364