[PATCH] D20560: [new PM] port LoopAccessAnalysis to new pass manager (part-1)

Adam Nemet via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 7 15:53:50 PDT 2016


anemet added a subscriber: anemet.

================
Comment at: lib/Transforms/Scalar/LoopDistribute.cpp:611
@@ -610,3 +610,3 @@
     // LAA will check that we only have a single exiting block.
-    LAI = &LAA->getInfo(L, ValueToValueMap());
+    LAI = &LAA->getInfo().getInfo(L, ValueToValueMap());
 
----------------
getInfo().getInfo(...), really?

What is the difference between LAFI and LAA conceptually?

Without knowing much about the new PM, I see that you split out the cache from LAA (which was pretty much all it contained).

Should this perhaps be:

LAA->getCache().getInfo(..)

or getLoopInfo(..)

?


http://reviews.llvm.org/D20560





More information about the llvm-commits mailing list