[PATCH] [Refactor] Replace RegionPasses by FunctionPasses

Tobias Grosser tobias at grosser.es
Sat Mar 7 14:40:23 PST 2015

The patch Zino refers to was committed in 231485. It did not have a large impact on our LNT performance builders (which do not seem to even see the slowdown we try to address here). However, on my laptop I was able to reproduce this performance issue in a cmake release build and Zino's patch fixes it at least for tramp3d-v4 nicely. Johannes, could you check if there is still a performance issue that needs to be addressed?

Comment at: lib/Transform/ScheduleOptimizer.cpp:58
@@ -57,3 +57,3 @@
-                 cl::desc("Dependences should be simplified (yes/no)"),
+                 cl::desc("DependenceInfo should be simplified (yes/no)"),
                  cl::Hidden, cl::init("yes"), cl::ZeroOrMore,
Why are these renamed?

Comment at: lib/Transform/ScheduleOptimizer.cpp:528
@@ -521,1 +527,3 @@
+  // constant coefficients will be removed from the DependenceInfo. The
+  // introduction of some additional DependenceInfo reduces the possible
   // transformations, but in most cases, such transformation do not seem to be
Is this rename intentional?



More information about the llvm-commits mailing list