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 patch to

[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

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 ubiz...@gmail.com wrote: On Thu, Jul 9, 2015 at 8:05 PM, H.J. Lu hongjiu...@intel.com 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

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 hongjiu...@intel.com 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