Re: [PATCH] rs6000: Improve rtx_costs for EQ a bit (PR30967)

2015-05-15 Thread Segher Boessenkool
On Thu, May 14, 2015 at 09:51:18AM -0400, David Edelsohn wrote: > On Wed, May 13, 2015 at 1:29 PM, Segher Boessenkool > wrote: > > On Wed, May 13, 2015 at 01:25:16PM -0400, David Edelsohn wrote: > >> > PR rtl-optimization/30967 > >> > * config/rs6000/rs6000.c (rs6000_rtx_costs): Do

Re: [PATCH] rs6000: Improve rtx_costs for EQ a bit (PR30967)

2015-05-14 Thread David Edelsohn
On Wed, May 13, 2015 at 1:29 PM, Segher Boessenkool wrote: > On Wed, May 13, 2015 at 01:25:16PM -0400, David Edelsohn wrote: >> > PR rtl-optimization/30967 >> > * config/rs6000/rs6000.c (rs6000_rtx_costs): Don't consider >> > destination mode for the cost of scc patterns. >

Re: [PATCH] rs6000: Improve rtx_costs for EQ a bit (PR30967)

2015-05-13 Thread Segher Boessenkool
On Wed, May 13, 2015 at 01:25:16PM -0400, David Edelsohn wrote: > > PR rtl-optimization/30967 > > * config/rs6000/rs6000.c (rs6000_rtx_costs): Don't consider > > destination mode for the cost of scc patterns. > > Okay. > > Should this be applied to GCC 5 branch? [ Testing

Re: [PATCH] rs6000: Improve rtx_costs for EQ a bit (PR30967)

2015-05-13 Thread David Edelsohn
On Wed, May 13, 2015 at 1:15 PM, Segher Boessenkool wrote: > This patch changes rs6000_rtx_costs to be closer to reality. It is > still not quite right, but at least it handles more cases, so the > > (set (reg:SI) (eq:SI (reg:SI) (reg:SI))) > > from the PR isn't taken as costing 1 insn an