[PATCH] Fix PR64829

Jiong Wang wong.kwongyuan.tools@gmail.com
Thu Jan 29 21:56:00 GMT 2015


2015-01-29 14:27 GMT+00:00 Kyrill Tkachov <kyrylo.tkachov@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



More information about the Gcc-patches mailing list