On Tue, Sep 13, 2016 at 5:29 PM, Segher Boessenkool <seg...@kernel.crashing.org> wrote: > On Tue, Sep 13, 2016 at 10:14:57AM -0500, Pat Haugen wrote: >> PRs 77536 and 68212 document cases where the estimated frequencies become >> unreliable to make appropriate decisions based on them. This patch removes >> the optimize_insn_for_speed_p check on the splitter so that the >> transformation will take place when -ffast-math is specified. >> >> Bootstrap/regtest on powerpc64le, ok for trunk? > > Okay for trunk. Please mention the PRs in the changelog, i.e.:
Maybe you instead want !optimize_insn_for_size then? (if such thing exists?) Otherwise you're triggering this for -Os. >> 2016-09-13 Pat Haugen <pthau...@us.ibm.com> >> > PR tree-optimization/77536 > PR rtl-optimization/68212 >> * config/rs6000/rs6000.md (div->recip splitter): Remove > > And you have trailing whitespace here ;-) > > Thanks, > > > Segher
