<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Wed, Jun 8, 2016 at 6:29 PM Xinliang David Li <<a href="mailto:xinliangli@gmail.com">xinliangli@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Jun 8, 2016 at 6:09 PM, Chandler Carruth 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">chandlerc added a comment.<br>
<br>
At a meta level, if I've started reviewing a patch, please wait until I have a chance to see your updates before submitting it. I'm sorry I didn't get to it in the early morning, but its still well under 24h turn around and I don't know that its reasonable to expect lower latency.<br>
<br>
That's not to say that I don't appreciate all of Adam and Davide's comments -- they're help reviewing is much appreciated -- but as it happens I don't actually think this is the right approach.<br></blockquote><div><br></div></div></div></div><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div>Ah -- you had initial comments saying this makes sense -- thus I assumed it is fine with lgtm.</div></div></div></div></blockquote><div><br></div><div>Sorry if I gave the wrong impression. =] At that point, I thought it did make sense, but I hadn't yet understood the nature of what LoopAccessInfo was doing. As it was cleaned up, it became much more clear.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><br></div><div>What you said below makes sense. I will rework on the patch to use LoopAnalysisManager to see if works.</div></div></div></div></blockquote><div><br></div><div>Yea, understand that this is a less well exercised part of the pass manager. I've essentially been going bottom-up to parallelize with other efforts. So you may run into some issues. Feel free to chat and get help if having trouble with the per-loop structuring of things.</div><div><br></div><div>-Chandler</div></div></div>