[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