[PATCH] D70734: [VPlan] Add basicblocks() and loop_basicblocks iterators.

[Gil Rapaport via Phabricator via llvm-commits]

gilr added a comment.

> Traversing VPRegions is not supported at the moment but we do not create such plans at the moment as far as I know

I assume you're referring to the native path? (non-native generates regions for scalarized & predicated instructions)



================
Comment at: llvm/lib/Transforms/Vectorize/VPlan.h:1404
+  /// the loop blocks.
+  iterator_range<vpbb_iterator_adaptor> basicblocks() {
+    VPRegionBlock *TopRegion = dyn_cast<VPRegionBlock>(Entry);
----------------
basicBlocks()?


================
Comment at: llvm/lib/Transforms/Vectorize/VPlan.h:1418
+                                 std::function<bool(VPBlockBase &)>>>
+  loop_basicblocks() {
+    VPRegionBlock *TopRegion = dyn_cast<VPRegionBlock>(Entry);
----------------
loopBasicBlocks()?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70734