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

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 2 11:38:30 PST 2019


fhahn added a comment.

In D70734#1764103 <https://reviews.llvm.org/D70734#1764103>, @gilr wrote:

> > 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)


Ah right, I missed that code path! For my initial use-case (dropping assumes), the iterator needs to traverse the regions as well then. Let's see if I can get the iterators to co-operate. Do you think it's worth to get the existing iterators in as is? There are a few places that could use them, I think.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70734





More information about the llvm-commits mailing list