[PATCH] D131989: [VPlan] Verify that header only contains header phi recipes.
Ayal Zaks via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Aug 21 03:10:32 PDT 2022
Ayal accepted this revision.
Ayal added a comment.
This revision is now accepted and ready to land.
Thanks for addressing, adding a couple of minor suggestions.
================
Comment at: llvm/lib/Transforms/Vectorize/VPlan.h:1137
static inline bool classof(const VPRecipeBase *B) {
return B->getVPDefID() == VPRecipeBase::VPCanonicalIVPHISC ||
B->getVPDefID() == VPRecipeBase::VPActiveLaneMaskPHISC ||
----------------
Phi-like recipes are kept in order to facilitate checking intervals (and help avoid such missed-case bugs)?
================
Comment at: llvm/lib/Transforms/Vectorize/VPlanVerifier.cpp:140
// Verify that phi-like recipes are at the beginning of the block, with no
// other recipes in between.
auto RecipeI = VPBB->begin();
----------------
Perhaps worth outlining the first part that verifies phi recipes, breaking the somewhat long method into two?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D131989/new/
https://reviews.llvm.org/D131989
More information about the llvm-commits
mailing list