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

Han-Kuan Chen via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 10 07:05:14 PST 2025


HanKuanChen 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)

Really appreciate you report. This is fixed by https://github.com/llvm/llvm-project/pull/122443/commits/5853a032bd078b15c1b7d0232452f129cb5e4c83 now.

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


More information about the llvm-commits mailing list