<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jun 7, 2016 at 3:53 PM, Adam Nemet via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">anemet added a subscriber: anemet.<br>
<br>
================<br>
Comment at: lib/Transforms/Scalar/LoopDistribute.cpp:611<br>
@@ -610,3 +610,3 @@<br>
     // LAA will check that we only have a single exiting block.<br>
-    LAI = &LAA->getInfo(L, ValueToValueMap());<br>
+    LAI = &LAA->getInfo().getInfo(L, ValueToValueMap());<br>
<br>
----------------<br>
getInfo().getInfo(...), really?<br></blockquote><div><br></div><div>yes -- that sounds kind of strange. I can make it similar to the new PM:</div><div><br></div><div>getResult().getInfo(..)</div><div><br></div><div>Do you have other preference?</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
What is the difference between LAFI and LAA conceptually?<br></blockquote><div><br></div><div><br></div><div>The difference is that LAA which is the legacy analysis pass will go away in the future, but LAFI will stay as cached analysis results maintained by new PM..</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Without knowing much about the new PM, I see that you split out the cache from LAA (which was pretty much all it contained).<br>
<br>
Should this perhaps be:<br>
<br>
LAA->getCache().getInfo(..)<br>
<br>
or getLoopInfo(..)<br>
<br></blockquote><div><br></div><div>getLoopInfo is not a good name as it has other meaning.  I can change it to getLoopAccessInfo, but I think getInfo should be fine if we change from LAA->getInfo() to LAA->getResult().</div><div><br></div><div>David</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
?<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
<a href="http://reviews.llvm.org/D20560" rel="noreferrer" target="_blank">http://reviews.llvm.org/D20560</a><br>
<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div></div>