<div dir="ltr">Long-overdue ping on this.<div><br></div><div>The aha test as it stands is an invalid C program since it accesses an array outside its bounds, though that data is never used due to a subsequent bounds check.  If we don't want to fix the test (given its third-party origin), should we at least add a comment documenting the invalid access?</div>
<div><br></div><div>Admittedly, there's almost no system that will crash on this code.  Previously, it was causing a failure in PNaCl-generated code, but a later PNaCl change (as a result of this investigation) fixed the failure.</div>
<div><br></div><div>Jim</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Jul 3, 2013 at 10:55 AM, Jim Stichnoth <span dir="ltr"><<a href="mailto:stichnot@google.com" target="_blank">stichnot@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div class="h5">On Wed, Jul 3, 2013 at 10:39 AM, Tim Northover <span dir="ltr"><<a href="mailto:t.p.northover@gmail.com" target="_blank">t.p.northover@gmail.com</a>></span> wrote:<br>
</div></div><div class="gmail_extra"><div class="gmail_quote"><div><div class="h5">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Jim,<br>
<div><br>
> The attached patch moves this clause to the beginning of the<br>
> condition, giving the same result without the illegal reference.<br>
<br>
</div>I think you forgot the patch.<br><br></blockquote></div></div><div>D'oh!  Thanks, attached now.</div><span class="HOEnZb"><font color="#888888"><div><br></div><div>Jim </div></font></span></div><br></div></div>

</blockquote></div><br></div>