Re: [PATCH][RTL] Fix PR87852

2018-11-05 Thread Richard Biener
On Fri, 2 Nov 2018, Eric Botcazou wrote: > > Bootstrapped on x86_64-unknown-linux-gnu, testing in progress. > > > > OK for trunk? > > > > Thanks, > > Richard. > > > > 2018-11-02 Richard Biener > > > > PR rtl-optimization/87852 > > * fwprop.c (use_killed_between): Only consider singl

Re: [PATCH][RTL] Fix PR87852

2018-11-02 Thread Jeff Law
On 11/2/18 11:12 AM, Eric Botcazou wrote: >> Bootstrapped on x86_64-unknown-linux-gnu, testing in progress. >> >> OK for trunk? >> >> Thanks, >> Richard. >> >> 2018-11-02 Richard Biener >> >> PR rtl-optimization/87852 >> * fwprop.c (use_killed_between): Only consider single-defs of the

Re: [PATCH][RTL] Fix PR87852

2018-11-02 Thread Jeff Law
On 11/2/18 7:40 AM, Richard Biener wrote: > > The following fixes PR87852, a latent bug in fwprop which when verifying > whether it may propagate a use from its definition site has a shortcut > > /* Check if the reg in USE has only one definition. We already > know that this definition re

Re: [PATCH][RTL] Fix PR87852

2018-11-02 Thread Eric Botcazou
> Bootstrapped on x86_64-unknown-linux-gnu, testing in progress. > > OK for trunk? > > Thanks, > Richard. > > 2018-11-02 Richard Biener > > PR rtl-optimization/87852 > * fwprop.c (use_killed_between): Only consider single-defs of the > use in the definition stmt that domina

[PATCH][RTL] Fix PR87852

2018-11-02 Thread Richard Biener
The following fixes PR87852, a latent bug in fwprop which when verifying whether it may propagate a use from its definition site has a shortcut /* Check if the reg in USE has only one definition. We already know that this definition reaches use, or we wouldn't be here. However, this