[PATCH] D129717: [VPlan] Initial def-use 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 following-up with this, good to see it didn't catch anything else, ship it!



================
Comment at: llvm/lib/Transforms/Vectorize/VPlanVerifier.cpp:194
+
+        // Skip blocks outside any region for now
+        auto *ParentR = VPBB->getParent();
----------------
nit: can early-continue/skip blocks in non-replicating regions.


================
Comment at: llvm/lib/Transforms/Vectorize/VPlanVerifier.cpp:233
   auto Iter = depth_first(
       VPBlockRecursiveTraversalWrapper<const VPBlockBase *>(Plan.getEntry()));
   for (const VPBasicBlock *VPBB :
----------------
nit: can fold the two dfs traversals into one - if a VPB hasn't had its BlockNumber assigned yet, it's number is/will-be greater than any which have.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129717