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 <rguent...@suse.de> >> >> PR rtl-optimization/87852 >> * fwprop.c (use_killed_between): Only consider single-defs of the >> use in the definition stmt that dominate it. > This looks OK to me, but this lacks commentary and I have a hard time parsing > the ChangeLog entry. Maybe: > > * fwprop.c (use_killed_between): Only consider single-defs of the use > whose definition statement dominates the use. > > FWIW I've attached a patch that also fixes the head comment of the function. LGTM. It does fix an armeb bug I was looking at from the c-torture suite. But sadly python still doesn't work :(
jeff
