[PATCH] [Refactor] Replace RegionPasses by FunctionPasses

[Tobias Grosser]

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 @@
     SimplifyDeps("polly-opt-simplify-deps",
-                 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?

http://reviews.llvm.org/D7986

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/