[llvm] [LV] Fixup tests following 93073af (PR #162091)
Ramkumar Ramachandra via llvm-commits
llvm-commits at lists.llvm.org
Mon Oct 6 07:22:31 PDT 2025
artagnon wrote:
> > I don't think updating the tests makes sense, if they now fail due to a verifier failure. There should be no verifier failures for valid input
>
> Nothing fails, and just some additional remarks are emitted?
Wait a sec: what does this mean?
"Phi-like recipe with different number of operands and predecessors."
It's a failure?
```cpp
if (auto *PhiIRI = dyn_cast<VPIRPhi>(&*RecipeI)) {
if (PhiIRI->getNumOperands() != VPBB->getNumPredecessors()) {
errs() << "Phi-like recipe with different number of operands and "
"predecessors.\n";
// TODO: Print broken recipe. At the moment printing an ill-formed
// phi-like recipe may crash.
return false;
}
}
```
Will revert the patch instead.
https://github.com/llvm/llvm-project/pull/162091
More information about the llvm-commits
mailing list