[PATCH] D111302: [VPlan] Add initial VPlan verification.

[Ayal Zaks via Phabricator via llvm-commits]

Ayal added a comment.

Looks good to me, thanks!
Perhaps also worth checking that all header-phi recipes reside in the header?



================
Comment at: llvm/lib/Transforms/Vectorize/VPlanVerifier.cpp:148
+      if (RecipeI->isPhi() && !isa<VPBlendRecipe>(&*RecipeI)) {
+        errs() << "Found phi-like recipe after non-phi recipes!\n";
+        return false;
----------------
worth spelling out the offenders?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D111302/new/

https://reviews.llvm.org/D111302