Re: [PATCH] fix PR translation/82185

2017-09-11 Thread Jeff Law
On 09/11/2017 03:59 PM, Max Filippov wrote: > Hi Richard, > > On Mon, Sep 11, 2017 at 2:36 PM, Richard Sandiford > wrote: >> Max Filippov writes: >>> 2017-09-11 Max Filippov >>> gcc/ >>> * expmed.c (emit_store_flag_int): Initialize rtx tem. >> >> LGTM, thanks, but I can't approve it. >>

Re: [PATCH] fix PR translation/82185

2017-09-11 Thread Max Filippov
Hi Richard, On Mon, Sep 11, 2017 at 2:36 PM, Richard Sandiford wrote: > Max Filippov writes: >> 2017-09-11 Max Filippov >> 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

Re: [PATCH] fix PR translation/82185

2017-09-11 Thread Richard Sandiford
Max Filippov writes: > 2017-09-11 Max Filippov > 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 spl

[PATCH] fix PR translation/82185

2017-09-11 Thread Max Filippov
2017-09-11 Max Filippov gcc/ * expmed.c (emit_store_flag_int): Initialize rtx tem. --- gcc/expmed.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gcc/expmed.c b/gcc/expmed.c index 7f0cb0a0ec05..945ab3d656a2 100644 --- a/gcc/expmed.c +++ b/gcc/expmed.c @@ -5601,7 +5