This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH, 4.9/4.10] Profile based option tuning


On Thu, Jul 24, 2014 at 3:52 AM, Pengfei Yuan <0xcoolypf@gmail.com> wrote:
> There are more.
>
> In toplev.c:
>   /* One region RA really helps to decrease the code size.  */
>   if (flag_ira_region == IRA_REGION_AUTODETECT)
>     flag_ira_region
>       = optimize_size || !optimize ? IRA_REGION_ONE : IRA_REGION_MIXED;

This could be fixed by moving this to ira.c

> In config/i386/i386.c:
>   * Assignment of ix86_cost
>   * Decision of alignment

True, I didn't grep backends.

Did you investigate where the savings come from?  I meanwhile fixed
the estimate_move_cost bit.

Thanks,
Richard.

> 2014-07-23 19:32 GMT+08:00 Richard Biener <richard.guenther@gmail.com>:
>> On Wed, Jul 23, 2014 at 1:04 PM, Pengfei Yuan <0xcoolypf@gmail.com> wrote:
>>> I guess some optimizations are controlled only by "optimize_size", not
>>> by the profile.
>>
>> I only see tree-inline.c:estimate_move_cost which we should indeed fix,
>> it could make a significant difference.
>>
>> One other use in tree-ssa-phiopt.c, but probably doesn't really matter.
>>
>> Richard.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]