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

[Ayal Zaks via Phabricator via llvm-commits]

Ayal accepted this revision.
Ayal added a comment.
This revision is now accepted and ready to land.

Thanks for pushing this forward!

Wondering about:

> /// Note that currently there is still an exception for VPBlendRecipes.



> // FIXME: At the moment, creating a mask might insert non-phi recipes before VPBlendRecipes.

VPBlendRecipes are indeed exceptional (not only //currently//?), as they represent a 'select' rather than a 'phi'. Is the //FIXME //needed?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111302