[PATCH] D53349: [VPlan] Changes to implement VPlan based predication for VPlan-native path.

Satish K Guggilla via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 16 16:25:31 PDT 2018


sguggill added inline comments.


================
Comment at: lib/Transforms/Vectorize/VPlanPredicator.cpp:266
+    : Plan(Plan), VPLI(&(Plan->getVPLoopInfo())) {
+  VPDomTree.recalculate(*(cast<VPRegionBlock>(Plan->getEntry())));
+}
----------------
Predicator is currently computing the dominator information for the top region. Once we start storing dominator information in a VPRegionBlock, we can avoid this recalculation.


Repository:
  rL LLVM

https://reviews.llvm.org/D53349





More information about the llvm-commits mailing list