[llvm] [LV] Move 3 functions into VPlanTransforms (NFC) (PR #158644)
Ramkumar Ramachandra via llvm-commits
llvm-commits at lists.llvm.org
Mon Oct 6 06:58:52 PDT 2025
artagnon wrote:
> > @artagnon looks there a test failures with expensive checks which looks related to the patch: https://lab.llvm.org/buildbot/#/builders/187/builds/12178.
> > Is it possible that some stale code got moved?
>
> Hm, that's strange: I thought there were no conflicts. Taking a look.
Ugh, it looks like runPass runs verifyVPlanIsValid when expensive checks are enabled:
```cpp
bool Res = Transform(Plan, Args...);
if (VerifyEachVPlan)
verifyVPlanIsValid(Plan);
return Res;
```
... should we just update the tests?
https://github.com/llvm/llvm-project/pull/158644
More information about the llvm-commits
mailing list