[PATCH, 4.9/4.10] Profile based option tuning

2014-07-20 Thread Pengfei Yuan
Hi, This patch tunes optimization options based on profile data: * Disable PGO options if profile is not available or empty. * Optimize for size if profile is available but empty. Here is an experiment on Firefox PGO build: CPU Intel Core i7-4770 RAM 32 GB

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

2014-07-20 Thread Pengfei Yuan
Sorry, tabs seems converted to spaces automatically. Please use the attachment instead. 2014-07-21 13:13 GMT+08:00 Pengfei Yuan <0xcool...@gmail.com>: > Hi, > > This patch tunes optimization options based on profile data: > * Disable PGO options if profile is not available or e

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

2014-07-22 Thread Pengfei Yuan
In the experiment, about 60% (1019/1699) profile data files are empty (all counters are zero). 2014-07-22 21:39 GMT+08:00 Richard Biener : > On Mon, Jul 21, 2014 at 7:13 AM, Pengfei Yuan <0xcool...@gmail.com> wrote: >> Hi, >> >> This patch tunes optimization op

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

2014-07-23 Thread Pengfei Yuan
14 at 2:39 AM, Pengfei Yuan <0xcool...@gmail.com> wrote: >> In the experiment, about 60% (1019/1699) profile data files are empty >> (all counters are zero). > > Well, but you are globally overriding options even for the parts with > profile. The whole point of profi

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

2014-07-23 Thread Pengfei Yuan
of alignment 2014-07-23 19:32 GMT+08:00 Richard Biener : > On Wed, Jul 23, 2014 at 1:04 PM, Pengfei Yuan <0xcool...@gmail.com> wrote: >> I guess some optimizations are controlled only by "optimize_size", not >> by the profile. > > I only see tree-inline.c:esti

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

2014-07-24 Thread Pengfei Yuan
No, I didn't. 2014-07-24 16:50 GMT+08:00 Richard Biener : > On Thu, Jul 24, 2014 at 3:52 AM, Pengfei Yuan <0xcool...@gmail.com> wrote: >> There are more. >> >> In toplev.c: >> /* One region RA really helps to decrease the code size. */ >>