[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