On Fri, Jun 23, 2017 at 9:06 AM, Bill Schmidt <wschm...@linux.vnet.ibm.com> wrote: > Hi, > > Here's version 2 of the patch to fix the missed SLSR PHI opportunities, > addressing Richard's comments. I've repeated regstrap and SPEC testing > on powerpc64le-unknown-linux-gnu, again showing the patch as neutral > with respect to performance. Is this ok for trunk? > > Thanks for the review! > > Bill > > > [gcc] > > 2016-06-23 Bill Schmidt <wschm...@linux.vnet.ibm.com> > > * gimple-ssa-strength-reduction.c (uses_consumed_by_stmt): New > function. > (find_basis_for_candidate): Call uses_consumed_by_stmt rather than > has_single_use. > (slsr_process_phi): Likewise. > (replace_uncond_cands_and_profitable_phis): Don't replace a > multiply candidate with a stride of 1 (copy or cast). > (phi_incr_cost): Call uses_consumed_by_stmt rather than > has_single_use. > (lowest_cost_path): Likewise. > (total_savings): Likewise. >
This may have caused: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81216 -- H.J.
