[llvm] [SLP] NFC. Replace MainOp and AltOp in TreeEntry with InstructionsState. (PR #120198)

Mikael Holmén via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 10 06:47:25 PST 2025


mikaelholmen wrote:

> Hi,
> 
> After recommit of this patch I still see the crash `` clang: ../lib/Transforms/Vectorize/SLPVectorizer.cpp:821: Instruction *(anonymous namespace)::InstructionsState::getMainOp() const: Assertion `valid() && "InstructionsState is invalid."' failed. `` for some downstream tests that https://lab.llvm.org/buildbot/#/builders/56/builds/15893 hit after the first version of this patch. I'll see if I can extract a reproducer but I'm not sure I'll manage before the weekend.

@HanKuanChen :
Reduced opt reproducer:
```opt -passes slp-vectorizer bbi-102823.ll -o /dev/null```
[bbi-102823.ll.gz](https://github.com/user-attachments/files/18378640/bbi-102823.ll.gz)


https://github.com/llvm/llvm-project/pull/120198


More information about the llvm-commits mailing list