[PATCH] D16382: Add LoopSimplifyCFG pass

escha via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 29 14:13:31 PST 2016


escha added inline comments.

================
Comment at: lib/Transforms/Scalar/LoopSimplifyCFG.cpp:70
@@ +69,3 @@
+INITIALIZE_PASS_DEPENDENCY(ScalarEvolutionWrapperPass)
+INITIALIZE_PASS_DEPENDENCY(TargetTransformInfoWrapperPass)
+INITIALIZE_PASS_DEPENDENCY(AssumptionCacheTracker)
----------------
sanjoy wrote:
> Why do you need to depend on `TargetTransformInfoWrapperPass` and `AssumptionCacheTracker`?
I guess I don't fully understand this section. Are these "dependencies" things we need to run, or are these "dependencies" things we can pass on to the next pass (preserved), or...? I notice a lot of passes seem to have things here that aren't listed as Required. How does this section relate to the Required/Preserves bit?


Repository:
  rL LLVM

http://reviews.llvm.org/D16382





More information about the llvm-commits mailing list