Re: [PATCH] Adjust variable shift costs for IA MCU

2015-08-24 Thread H.J. Lu
On Thu, Jul 09, 2015 at 11:05:27AM -0700, H.J. Lu wrote: > We reduce code size for IA MCU by adjusting variable shift costs for IA > MCU. OK for trunk? > > Thanks. > > > H.J. > --- > PR target/66821 > * config/i386/i386.c (iamcu_cost): Adjust variable shift costs. I checked in this

Re: [PATCH] Adjust variable shift costs for IA MCU

2015-07-09 Thread H.J. Lu
On Thu, Jul 9, 2015 at 1:30 PM, Uros Bizjak wrote: > On Thu, Jul 9, 2015 at 8:05 PM, H.J. Lu wrote: >> We reduce code size for IA MCU by adjusting variable shift costs for IA >> MCU. OK for trunk? > > IMO, tuning patches should fall into "obvious" category. I don't have > any data to to do any m

Re: [PATCH] Adjust variable shift costs for IA MCU

2015-07-09 Thread Uros Bizjak
On Thu, Jul 9, 2015 at 8:05 PM, H.J. Lu wrote: > We reduce code size for IA MCU by adjusting variable shift costs for IA > MCU. OK for trunk? IMO, tuning patches should fall into "obvious" category. I don't have any data to to do any meaningful review of a cost metric for a new target. So, inst

[PATCH] Adjust variable shift costs for IA MCU

2015-07-09 Thread H.J. Lu
We reduce code size for IA MCU by adjusting variable shift costs for IA MCU. OK for trunk? Thanks. H.J. --- PR target/66821 * config/i386/i386.c (iamcu_cost): Adjust variable shift costs. --- gcc/config/i386/i386.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --g