[PATCH] D101093: [VPlan] Add VPBlockUtils::basicBlocksOnly helper.

Andrei Elovikov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 22 12:21:55 PDT 2021


a.elovikov accepted this revision.
a.elovikov added a comment.
This revision is now accepted and ready to land.

VPlan part looks good to me. I'm not familiar with implementation details of the `filtered_iterator` to discuss the `to_reference->filter->to_pointer` approach though. Regardless of that, it's an improvement in interfaces, so LGTM unless someone has a way to make filtering work with pointer directly.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101093



More information about the llvm-commits mailing list