On 18/09/2015 14:26, "Alan Lawrence" <alan.lawre...@arm.com> wrote:

>On 18/09/15 13:17, Richard Biener wrote:
>>
>> Ok, I see.
>>
>> That this case is already vectorized is because it implements MAX_EXPR,
>> modifying it slightly to
>>
>> int foo (int *a)
>> {
>>    int val = 0;
>>    for (int i = 0; i < 1024; ++i)
>>      if (a[i] > val)
>>        val = a[i] + 1;
>>    return val;
>> }
>>
>> makes it no longer handled by current code.
>>
>
>Yes. I believe the idea for the patch is to handle arbitrary expressions
>like
>
>int foo (int *a)
>{
>    int val = 0;
>    for (int i = 0; i < 1024; ++i)
>      if (some_expression (i))
>        val = another_expression (i);
>    return val;
>}


Yes, that’s correct. Hopefully my new test cases should cover everything.


Alan.


Reply via email to