Re: Drop frequencies from basic blocks

2017-11-06 Thread Markus Trippelsdorf
On 2017.11.07 at 00:12 +0100, Jan Hubicka wrote: > > On 2017.11.05 at 11:55 +0100, Jan Hubicka wrote: > > > > On 2017.11.03 at 16:48 +0100, Jan Hubicka wrote: > > > > > this is updated patch which I have comitted after > > > > > profiledbootstrapping x86-64 > > > > > > > > Unfortunately, compiling

Re: Drop frequencies from basic blocks

2017-11-06 Thread Jan Hubicka
> On 2017.11.05 at 11:55 +0100, Jan Hubicka wrote: > > > On 2017.11.03 at 16:48 +0100, Jan Hubicka wrote: > > > > this is updated patch which I have comitted after profiledbootstrapping > > > > x86-64 > > > > > > Unfortunately, compiling tramp3d-v4.cpp is 6-7% slower after this patch. > > > This

Re: Drop frequencies from basic blocks

2017-11-05 Thread Markus Trippelsdorf
On 2017.11.05 at 11:55 +0100, Jan Hubicka wrote: > > On 2017.11.03 at 16:48 +0100, Jan Hubicka wrote: > > > this is updated patch which I have comitted after profiledbootstrapping > > > x86-64 > > > > Unfortunately, compiling tramp3d-v4.cpp is 6-7% slower after this patch. > > This happens with a

Re: Drop frequencies from basic blocks

2017-11-05 Thread Jan Hubicka
> On 2017.11.03 at 16:48 +0100, Jan Hubicka wrote: > > this is updated patch which I have comitted after profiledbootstrapping > > x86-64 > > Unfortunately, compiling tramp3d-v4.cpp is 6-7% slower after this patch. > This happens with an LTO/PGO bootstrapped gcc using --enable-checking=release.

Re: Drop frequencies from basic blocks

2017-11-05 Thread Markus Trippelsdorf
On 2017.11.03 at 16:48 +0100, Jan Hubicka wrote: > this is updated patch which I have comitted after profiledbootstrapping x86-64 Unfortunately, compiling tramp3d-v4.cpp is 6-7% slower after this patch. This happens with an LTO/PGO bootstrapped gcc using --enable-checking=release. On X86_64: Bef

Re: Drop frequencies from basic blocks

2017-11-04 Thread Andreas Schwab
This breaks gfortran.dg/bounds_check_15.f90 with -O3 -funroll-loops -fbounds-check on ia64: Error: qsort comparator non-negative on sorted output: 1 during RTL pass: mach ../gcc/testsuite/gfortran.dg/bounds_check_15.f90:32:0: internal compiler error: qsort checking failed 0x401cc1ff qsort

Re: Drop frequencies from basic blocks

2017-11-02 Thread Jan Hubicka
> Can be also seen in GCC PGO: > > checking for ssize_t... ../../libdecnumber/decNumber.c: In function > ‘decDecap’: > ../../libdecnumber/decNumber.c:7640:25: error: corrupted value profile: > interval profile counter (0 out of 0) inconsistent with basic-block count > (4356) >if (cut!=DECDP

Re: Drop frequencies from basic blocks

2017-11-02 Thread Martin Liška
Apart from that I also see this GCC PGO: ../../libiberty/pex-unix.c:789:1: warning: Missing counts for called function pex_child_error.isra.1/69 } ^ during IPA pass: inline ../../libiberty/pex-unix.c: In function ‘pex_child_error.isra.1’: ../../libiberty/pex-unix.c:373:1: internal compiler er

Re: Drop frequencies from basic blocks

2017-11-02 Thread Martin Liška
Can be also seen in GCC PGO: checking for ssize_t... ../../libdecnumber/decNumber.c: In function ‘decDecap’: ../../libdecnumber/decNumber.c:7640:25: error: corrupted value profile: interval profile counter (0 out of 0) inconsistent with basic-block count (4356) if (cut!=DECDPUN) *msu%=powers[

Re: Drop frequencies from basic blocks

2017-11-02 Thread Martin Liška
On 11/02/2017 08:06 PM, Jan Hubicka wrote: Sorry, I must have used older diff file, because it is one of unfinished chnages I made today. I am attaching correct diff. Thank you. This one works for me, however I see various errors for postgres PGO: cd src/backend/replication/ marxin@marxinbox

Re: Drop frequencies from basic blocks

2017-11-02 Thread Martin Liška
Hi Honza. Thanks for the huge patch. I'm willing to help you with testing, but I can't apply the patch on top of r254348: ../../gcc/profile.c: In function ‘void compute_branch_probabilities(unsigned int, unsigned int)’: ../../gcc/profile.c:794:11: error: ‘flag_guess_branch_probability’ was not