2015-01-29 14:27 GMT+00:00 Kyrill Tkachov <kyrylo.tkac...@arm.com>:
>
> On 29/01/15 12:44, Richard Biener wrote:
>>
>>
>> Can you test?
>
two issues during bootstrapping the patch on arm board on top of r220249
* one warning treated as error.
declaration of "new_stmt" need to be removed in
vect_handle_widen_op_by_const.
* bootstrap failed, because of binary differs.
Not sure it's caused by this patch as I haven't bootstrap r220249
without this patch, will start a night build to verify.
Regards,
Jiong
>
> This version fixes the ICE in the PR and the testsuite runs looks ok too.
> Minor nit, the definition of new_stmt in vect_handle_widen_op_by_const
> should be removed to avoid an 'unused variable' warning.
> Jiong (or somebody else), could you please bootstrap this on arm when you
> get the chance? I'll be offline in the next couple of days.
>
> Thanks,
> Kyrill
