[PATCH] fix PR translation/82185
Max Filippov
jcmvbkbc@gmail.com
Mon Sep 11 21:59:00 GMT 2017
Hi Richard,
On Mon, Sep 11, 2017 at 2:36 PM, Richard Sandiford
<richard.sandiford@linaro.org> wrote:
> Max Filippov <jcmvbkbc@gmail.com> writes:
>> 2017-09-11 Max Filippov <jcmvbkbc@gmail.com>
>> gcc/
>> * expmed.c (emit_store_flag_int): Initialize rtx tem.
>
> LGTM, thanks, but I can't approve it.
>
> This makes the later "tem = 0;" redundant, so perhaps it would make
> sense to delete that too? There again, it was redundant before the
> split as well.
>
> An alternative would be to only test tem when we've done something
> with it, as below, but I don't know if that's better or a step backwards.
this works for me too, so whichever fix you like better.
--
Thanks.
-- Max
More information about the Gcc-patches
mailing list