I went down to Austin and did a walkthrough of the code with (generous!) folks from Qualcomm. As a result, I have a long list of changes I'd like to make. Therefore, you might hold off any further review. I'll submit an updated version when it's ready (probably a week or two).<div>
<br></div><div>Thanks,</div><div>Preston</div><div><br><br><div class="gmail_quote">On Sun, Aug 12, 2012 at 4:39 PM, Andrew Trick <span dir="ltr"><<a href="mailto:atrick@apple.com" target="_blank">atrick@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Aug 10, 2012, at 4:13 PM, Hal Finkel <<a href="mailto:hfinkel@anl.gov">hfinkel@anl.gov</a>> wrote:<br>
<br>
> On Thu, 9 Aug 2012 11:25:13 -0700<br>
> Preston Briggs <<a href="mailto:preston.briggs@gmail.com">preston.briggs@gmail.com</a>> wrote:<br>
><br>
>> Foo! Trying again, with the promised attachment.<br>
>><br>
>> Preston<br>
>><br>
>><br>
>> On Thu, Aug 9, 2012 at 11:23 AM, Preston Briggs<br>
>> <<a href="mailto:preston.briggs@gmail.com">preston.briggs@gmail.com</a>> wrote:<br>
>>> Hal,<br>
>>><br>
>>> Here's an updated patch file for the dependence-analysis pass.<br>
>>> This version includes the GCD test (our first MIV test),<br>
>>> plus a few new test cases.<br>
><br>
> Great! I'll try to look at this soon -- others should too; don't wait on<br>
> me ;)<br>
><br>
> -Hal<br>
<br>
</div>I also plan to review this, but only after getting back from vacation and doing some catching up. So don't wait for me either. Hopefully someone else beats me to it<br>
<br>
Andy<br>
</blockquote></div><br></div>