[PATCH] D115750: [SLP]Further improvement of the cost model for scalars used in buildvectors.

Mon May 9 11:53:36 PDT 2022

sberg added a comment.

In D115750#3500827 <https://reviews.llvm.org/D115750#3500827>, @ABataev wrote:

> Must be fixed in 9c3a75eabf577f0e0e372be95ec4861600a5acdb <https://reviews.llvm.org/rG9c3a75eabf577f0e0e372be95ec4861600a5acdb>

still fails for me, see https://github.com/llvm/llvm-project/issues/55359 "'llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:6596: llvm::InstructionCost llvm::slpvectorizer::BoUpSLP::getTreeCost(ArrayRef<llvm::Value *>): Assertion `Mask[InIdx] == UndefMaskElem && "InsertElementInstruction used already."' failed.' since https://github.com/llvm/llvm-project/commit/99f31acfce338417fea3c14983d6f8fedc8ed043 '[SLP]Further improvement of the cost model for scalars used in buildvectors.'" (which I filed so that I could attach the non-reduced reproducer there)

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115750