[PATCH] D22437: [LoopDist] Port to new PM

Sean Silva via llvm-commits llvm-commits at lists.llvm.org
Sat Jul 16 15:39:10 PDT 2016


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

One small comment but this LGTM.


================
Comment at: lib/Transforms/Scalar/LoopDistribute.cpp:962
@@ +961,3 @@
+
+  for (Loop *TopLevelLoop : LI)
+    for (Loop *L : depth_first(TopLevelLoop))
----------------
Usually we have been factoring out a "runImpl" to be shared by the old PM and the new PM. Is that doable here to avoid some of the duplication?


https://reviews.llvm.org/D22437





More information about the llvm-commits mailing list