[PATCH] Fix the broken aha.c test
Jim Stichnoth
stichnot at google.com
Mon Sep 16 15:27:28 PDT 2013
Long-overdue ping on this.
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?
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.
Jim
On Wed, Jul 3, 2013 at 10:55 AM, Jim Stichnoth <stichnot at google.com> wrote:
> On Wed, Jul 3, 2013 at 10:39 AM, Tim Northover <t.p.northover at gmail.com>wrote:
>
>> Hi Jim,
>>
>> > The attached patch moves this clause to the beginning of the
>> > condition, giving the same result without the illegal reference.
>>
>> I think you forgot the patch.
>>
>> D'oh! Thanks, attached now.
>
> Jim
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130916/c3ce1b16/attachment.html>
More information about the llvm-commits
mailing list