[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