* [Bug bootstrap/46528] [ia64] profiledbootstrap failure
2010-11-17 23:35 [Bug bootstrap/46528] New: [ia64] profiledbootstrap failure steven at gcc dot gnu.org
@ 2010-11-18 10:11 ` rguenth at gcc dot gnu.org
2010-11-18 15:03 ` mkuvyrkov at gcc dot gnu.org
` (21 subsequent siblings)
22 siblings, 0 replies; 25+ messages in thread
From: rguenth at gcc dot gnu.org @ 2010-11-18 10:11 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46528
--- Comment #1 from Richard Guenther <rguenth at gcc dot gnu.org> 2010-11-18 10:09:44 UTC ---
What you do is look at the gimple before pass_ipa_tree_profile at profile-use
and profile-generate time, they should have matching CFGs. If they don't
then that's the bug.
^ permalink raw reply [flat|nested] 25+ messages in thread
* [Bug bootstrap/46528] [ia64] profiledbootstrap failure
2010-11-17 23:35 [Bug bootstrap/46528] New: [ia64] profiledbootstrap failure steven at gcc dot gnu.org
2010-11-18 10:11 ` [Bug bootstrap/46528] " rguenth at gcc dot gnu.org
@ 2010-11-18 15:03 ` mkuvyrkov at gcc dot gnu.org
2010-11-18 22:09 ` steven at gcc dot gnu.org
` (20 subsequent siblings)
22 siblings, 0 replies; 25+ messages in thread
From: mkuvyrkov at gcc dot gnu.org @ 2010-11-18 15:03 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46528
Maxim Kuvyrkov <mkuvyrkov at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |mkuvyrkov at gcc dot
| |gnu.org
--- Comment #2 from Maxim Kuvyrkov <mkuvyrkov at gcc dot gnu.org> 2010-11-18 14:55:35 UTC ---
Also confirmed on x86_64-unknown-linux-gnu. The execution numbers are exactly
the same as on ia64.
^ permalink raw reply [flat|nested] 25+ messages in thread
* [Bug bootstrap/46528] [ia64] profiledbootstrap failure
2010-11-17 23:35 [Bug bootstrap/46528] New: [ia64] profiledbootstrap failure steven at gcc dot gnu.org
2010-11-18 10:11 ` [Bug bootstrap/46528] " rguenth at gcc dot gnu.org
2010-11-18 15:03 ` mkuvyrkov at gcc dot gnu.org
@ 2010-11-18 22:09 ` steven at gcc dot gnu.org
2010-11-20 23:19 ` [Bug bootstrap/46528] [4.6 Regression] " ubizjak at gmail dot com
` (19 subsequent siblings)
22 siblings, 0 replies; 25+ messages in thread
From: steven at gcc dot gnu.org @ 2010-11-18 22:09 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46528
Steven Bosscher <steven at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|UNCONFIRMED |NEW
Last reconfirmed| |2010.11.18 21:43:15
Ever Confirmed|0 |1
--- Comment #3 from Steven Bosscher <steven at gcc dot gnu.org> 2010-11-18 21:43:15 UTC ---
Also happens with --enable-checking=release on ia64:
../../trunk/libcpp/macro.c: In function 'cpp_macro_definition':
../../trunk/libcpp/macro.c:2123:1: error: corrupted profile info: profile data
is not flow-consistent
../../trunk/libcpp/macro.c:2123:1: error: corrupted profile info: number of
executions for edge 45-59 thought to be -104
../../trunk/libcpp/macro.c:2123:1: error: corrupted profile info: number of
executions for edge 45-1 thought to be 104
../../trunk/libcpp/macro.c:2123:1: error: corrupted profile info: number of
executions for edge 55-56 thought to be 10
../../trunk/libcpp/macro.c:2123:1: error: corrupted profile info: number of
executions for edge 55-57 thought to be -5
make[3]: *** [macro.o] Error 1
make[3]: *** Waiting for unfinished jobs....
make[3]: Leaving directory `/home/stevenb/devel/build-test/libcpp'
make[2]: *** [all-stagefeedback-libcpp] Error 2
make[2]: Leaving directory `/home/stevenb/devel/build-test'
make[1]: *** [stagefeedback-bubble] Error 2
make[1]: Leaving directory `/home/stevenb/devel/build-test'
make: *** [profiledbootstrap] Error 2
stevenb@iowa:~/devel/build-test$ cat gcc/configargs.h
/* Generated automatically. */
static const char configuration_arguments[] = "../trunk/configure
--with-gmp=/home/stevenb --with-mpfr=/home/stevenb --enable-languages=c
--enable-checking=release --disable-nls --disable-libmudflap --disable-libssp";
static const char thread_model[] = "posix";
^ permalink raw reply [flat|nested] 25+ messages in thread
* [Bug bootstrap/46528] [4.6 Regression] profiledbootstrap failure
2010-11-17 23:35 [Bug bootstrap/46528] New: [ia64] profiledbootstrap failure steven at gcc dot gnu.org
` (2 preceding siblings ...)
2010-11-18 22:09 ` steven at gcc dot gnu.org
@ 2010-11-20 23:19 ` ubizjak at gmail dot com
2010-11-21 20:08 ` hjl.tools at gmail dot com
` (18 subsequent siblings)
22 siblings, 0 replies; 25+ messages in thread
From: ubizjak at gmail dot com @ 2010-11-20 23:19 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46528
Uros Bizjak <ubizjak at gmail dot com> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |ubizjak at gmail dot com
Target Milestone|--- |4.6.0
Summary|[ia64] profiledbootstrap |[4.6 Regression]
|failure |profiledbootstrap failure
--- Comment #4 from Uros Bizjak <ubizjak at gmail dot com> 2010-11-20 23:00:47 UTC ---
Also on alpha, you will need [1] to get this far.
Using built-in specs.
COLLECT_GCC=./prev-gcc/xgcc
Target: alphaev68-unknown-linux-gnu
Configured with: ../gcc-svn/trunk/configure --enable-languages=c,c++,fortran
Thread model: posix
gcc version 4.6.0 20101120 (experimental) [trunk revision 166974] (GCC)
Marked as 4.6 regression.
[1] http://gcc.gnu.org/ml/gcc-patches/2010-11/msg02118.html
^ permalink raw reply [flat|nested] 25+ messages in thread
* [Bug bootstrap/46528] [4.6 Regression] profiledbootstrap failure
2010-11-17 23:35 [Bug bootstrap/46528] New: [ia64] profiledbootstrap failure steven at gcc dot gnu.org
` (3 preceding siblings ...)
2010-11-20 23:19 ` [Bug bootstrap/46528] [4.6 Regression] " ubizjak at gmail dot com
@ 2010-11-21 20:08 ` hjl.tools at gmail dot com
2010-11-24 21:08 ` ubizjak at gmail dot com
` (17 subsequent siblings)
22 siblings, 0 replies; 25+ messages in thread
From: hjl.tools at gmail dot com @ 2010-11-21 20:08 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46528
H.J. Lu <hjl.tools at gmail dot com> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |hjl.tools at gmail dot com,
| |rguenth at gcc dot gnu.org
--- Comment #5 from H.J. Lu <hjl.tools at gmail dot com> 2010-11-21 20:00:33 UTC ---
It is caused by revision revision 164986:
http://gcc.gnu.org/ml/gcc-cvs/2010-10/msg00168.html
^ permalink raw reply [flat|nested] 25+ messages in thread
* [Bug bootstrap/46528] [4.6 Regression] profiledbootstrap failure
2010-11-17 23:35 [Bug bootstrap/46528] New: [ia64] profiledbootstrap failure steven at gcc dot gnu.org
` (4 preceding siblings ...)
2010-11-21 20:08 ` hjl.tools at gmail dot com
@ 2010-11-24 21:08 ` ubizjak at gmail dot com
2010-11-24 21:36 ` ubizjak at gmail dot com
` (16 subsequent siblings)
22 siblings, 0 replies; 25+ messages in thread
From: ubizjak at gmail dot com @ 2010-11-24 21:08 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46528
--- Comment #6 from Uros Bizjak <ubizjak at gmail dot com> 2010-11-24 21:00:02 UTC ---
Created attachment 22514
--> http://gcc.gnu.org/bugzilla/attachment.cgi?id=22514
preprocessed file
^ permalink raw reply [flat|nested] 25+ messages in thread
* [Bug bootstrap/46528] [4.6 Regression] profiledbootstrap failure
2010-11-17 23:35 [Bug bootstrap/46528] New: [ia64] profiledbootstrap failure steven at gcc dot gnu.org
` (5 preceding siblings ...)
2010-11-24 21:08 ` ubizjak at gmail dot com
@ 2010-11-24 21:36 ` ubizjak at gmail dot com
2010-11-24 21:51 ` rguenth at gcc dot gnu.org
` (15 subsequent siblings)
22 siblings, 0 replies; 25+ messages in thread
From: ubizjak at gmail dot com @ 2010-11-24 21:36 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46528
--- Comment #7 from Uros Bizjak <ubizjak at gmail dot com> 2010-11-24 21:01:16 UTC ---
Created attachment 22516
--> http://gcc.gnu.org/bugzilla/attachment.cgi?id=22516
corresponding profile
~/gcc-build/prev-gcc/cc1 -g -O2 -fprofile-use -quiet macro.i
../../gcc-svn/trunk/libcpp/macro.c: In function ‘cpp_macro_definition’:
../../gcc-svn/trunk/libcpp/macro.c:2123:1: error: corrupted profile info:
profile data is not flow-consistent
../../gcc-svn/trunk/libcpp/macro.c:2123:1: error: corrupted profile info:
number of executions for edge 45-59 thought to be -104
../../gcc-svn/trunk/libcpp/macro.c:2123:1: error: corrupted profile info:
number of executions for edge 45-1 thought to be 104
../../gcc-svn/trunk/libcpp/macro.c:2123:1: error: corrupted profile info:
number of executions for edge 55-56 thought to be 10
../../gcc-svn/trunk/libcpp/macro.c:2123:1: error: corrupted profile info:
number of executions for edge 55-57 thought to be -5
^ permalink raw reply [flat|nested] 25+ messages in thread
* [Bug bootstrap/46528] [4.6 Regression] profiledbootstrap failure
2010-11-17 23:35 [Bug bootstrap/46528] New: [ia64] profiledbootstrap failure steven at gcc dot gnu.org
` (6 preceding siblings ...)
2010-11-24 21:36 ` ubizjak at gmail dot com
@ 2010-11-24 21:51 ` rguenth at gcc dot gnu.org
2010-11-24 22:04 ` rguenth at gcc dot gnu.org
` (14 subsequent siblings)
22 siblings, 0 replies; 25+ messages in thread
From: rguenth at gcc dot gnu.org @ 2010-11-24 21:51 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46528
--- Comment #8 from Richard Guenther <rguenth at gcc dot gnu.org> 2010-11-24 21:18:43 UTC ---
I can't reproduce it on x86_64-unknown-linux-gnu with
./configure --disable-checking --enable-languages=c
or with
./configure --enable-languages=c,c++,fortran
I can reproduce it with the attached testcase and stage1 xgcc though:
> ./xgcc -B. -O2 -g -fprofile-use -c macro.i
../../gcc-svn/trunk/libcpp/macro.c: In function 'cpp_macro_definition':
../../gcc-svn/trunk/libcpp/macro.c:2123:1: error: corrupted profile info:
profile data is not flow-consistent
../../gcc-svn/trunk/libcpp/macro.c:2123:1: error: corrupted profile info:
number of executions for edge 45-59 thought to be -104
../../gcc-svn/trunk/libcpp/macro.c:2123:1: error: corrupted profile info:
number of executions for edge 45-1 thought to be 104
../../gcc-svn/trunk/libcpp/macro.c:2123:1: error: corrupted profile info:
number of executions for edge 55-56 thought to be 10
../../gcc-svn/trunk/libcpp/macro.c:2123:1: error: corrupted profile info:
number of executions for edge 55-57 thought to be -5
would be nice to know if
BOOT_CFLAGS="-O2 -g -fno-inline" ./configure --enable-languages=c
make profiledbootstrap BOOT_CFLAGS="-O2 -g -fno-inline"
also fails, alternatively s/-g -fno-inline/-g0/ or
s/-fno-inline/-fno-partial-inlining.
^ permalink raw reply [flat|nested] 25+ messages in thread
* [Bug bootstrap/46528] [4.6 Regression] profiledbootstrap failure
2010-11-17 23:35 [Bug bootstrap/46528] New: [ia64] profiledbootstrap failure steven at gcc dot gnu.org
` (7 preceding siblings ...)
2010-11-24 21:51 ` rguenth at gcc dot gnu.org
@ 2010-11-24 22:04 ` rguenth at gcc dot gnu.org
2010-11-24 22:17 ` ubizjak at gmail dot com
` (13 subsequent siblings)
22 siblings, 0 replies; 25+ messages in thread
From: rguenth at gcc dot gnu.org @ 2010-11-24 22:04 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46528
--- Comment #9 from Richard Guenther <rguenth at gcc dot gnu.org> 2010-11-24 21:50:55 UTC ---
The CFGs match, the spanning tree as well. Instrumenting needs to split
two edges, but that shouldn't matter and happens elsewhere as well.
But we have in .optimized (of the -fprofile-generate case)
<bb 11>:
pretmp.676_490 = (unsigned int) D.8874_415;
pretmp.677_491 = macro_39->params;
pretmp.681_496 = *.LPBX1[67];
<bb 12>:
# len_429 = PHI <len_55(12), len_44(11)>
# ivtmp.698_171 = PHI <ivtmp.698_92(12), 0(11)>
D.8871_52 = MEM[base: pretmp.677_491, index: ivtmp.698_171, step: 8, offset:
0B];
D.8872_53 = D.8871_52->ident.len;
D.8873_54 = D.8872_53 + 1;
len_55 = D.8873_54 + len_429;
# DEBUG len => len_55
# DEBUG i => D#11
# DEBUG len => len_55
# DEBUG i => D#11
ivtmp.698_92 = ivtmp.698_171 + 1;
i_64 = (unsigned int) ivtmp.698_92;
if (i_64 < pretmp.676_490)
goto <bb 12>;
else
goto <bb 13>;
<bb 13>:
D.11695_474 = pretmp.681_496 + 1;
D.11697_493 = pretmp.676_490 + 4294967295;
D.11698_127 = (long int) D.11697_493;
*.LPBX1_I_lsm.686_486 = D.11695_474 + D.11698_127;
*.LPBX1[67] = *.LPBX1_I_lsm.686_486;
which at least looks odd to me and produced by SCCP (supposedly
*.LPBX1[67] + 1 + (long)(niter - 1), so not 100% invalid).
Soooo ... for those who can reproduce it, can you try with
-fno-tree-scev-cprop?
I'll wrap my brain around that above computation tomorrow ;)
^ permalink raw reply [flat|nested] 25+ messages in thread
* [Bug bootstrap/46528] [4.6 Regression] profiledbootstrap failure
2010-11-17 23:35 [Bug bootstrap/46528] New: [ia64] profiledbootstrap failure steven at gcc dot gnu.org
` (8 preceding siblings ...)
2010-11-24 22:04 ` rguenth at gcc dot gnu.org
@ 2010-11-24 22:17 ` ubizjak at gmail dot com
2010-11-24 22:32 ` ubizjak at gmail dot com
` (12 subsequent siblings)
22 siblings, 0 replies; 25+ messages in thread
From: ubizjak at gmail dot com @ 2010-11-24 22:17 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46528
--- Comment #10 from Uros Bizjak <ubizjak at gmail dot com> 2010-11-24 22:10:11 UTC ---
Created attachment 22517
--> http://gcc.gnu.org/bugzilla/attachment.cgi?id=22517
-fno-inline preprocessed source
^ permalink raw reply [flat|nested] 25+ messages in thread
* [Bug bootstrap/46528] [4.6 Regression] profiledbootstrap failure
2010-11-17 23:35 [Bug bootstrap/46528] New: [ia64] profiledbootstrap failure steven at gcc dot gnu.org
` (9 preceding siblings ...)
2010-11-24 22:17 ` ubizjak at gmail dot com
@ 2010-11-24 22:32 ` ubizjak at gmail dot com
2010-11-24 22:57 ` ubizjak at gmail dot com
` (11 subsequent siblings)
22 siblings, 0 replies; 25+ messages in thread
From: ubizjak at gmail dot com @ 2010-11-24 22:32 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46528
--- Comment #11 from Uros Bizjak <ubizjak at gmail dot com> 2010-11-24 22:12:46 UTC ---
Created attachment 22518
--> http://gcc.gnu.org/bugzilla/attachment.cgi?id=22518
corresponding -fno-inline profile
~/gcc-build/prev-gcc/cc1 -g -O2 -fno-inline -fprofile-use -quiet macro.i
../../gcc-svn/trunk/libcpp/macro.c: In function ‘cpp_macro_definition’:
../../gcc-svn/trunk/libcpp/macro.c:2123:1: error: corrupted profile info:
profile data is not flow-consistent
../../gcc-svn/trunk/libcpp/macro.c:2123:1: error: corrupted profile info:
number of executions for edge 45-59 thought to be -104
../../gcc-svn/trunk/libcpp/macro.c:2123:1: error: corrupted profile info:
number of executions for edge 45-1 thought to be 104
../../gcc-svn/trunk/libcpp/macro.c:2123:1: error: corrupted profile info:
number of executions for edge 55-56 thought to be 10
../../gcc-svn/trunk/libcpp/macro.c:2123:1: error: corrupted profile info:
number of executions for edge 55-57 thought to be -5
^ permalink raw reply [flat|nested] 25+ messages in thread
* [Bug bootstrap/46528] [4.6 Regression] profiledbootstrap failure
2010-11-17 23:35 [Bug bootstrap/46528] New: [ia64] profiledbootstrap failure steven at gcc dot gnu.org
` (10 preceding siblings ...)
2010-11-24 22:32 ` ubizjak at gmail dot com
@ 2010-11-24 22:57 ` ubizjak at gmail dot com
2010-11-24 23:06 ` ubizjak at gmail dot com
` (10 subsequent siblings)
22 siblings, 0 replies; 25+ messages in thread
From: ubizjak at gmail dot com @ 2010-11-24 22:57 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46528
--- Comment #12 from Uros Bizjak <ubizjak at gmail dot com> 2010-11-24 22:17:13 UTC ---
glibc data form F14:
./libc.so.6
GNU C Library development release version 2.12.90, by Roland McGrath et al.
Copyright (C) 2010 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.
There is NO warranty; not even for MERCHANTABILITY or FITNESS FOR A
PARTICULAR PURPOSE.
Compiled by GNU CC version 4.5.1 20100924 (Red Hat 4.5.1-4).
Compiled on a Linux 2.6.35 system on 2010-11-12.
Available extensions:
Support for some architectures added on, not maintained in glibc core.
The C stubs add-on version 2.1.2.
crypt add-on version 2.1 by Michael Glad and others
GNU Libidn by Simon Josefsson
Native POSIX Threads Library by Ulrich Drepper et al
BIND-8.2.3-T5B
RT using linux kernel aio
libc ABIs: UNIQUE IFUNC
For bug reporting instructions, please see:
<http://www.gnu.org/software/libc/bugs.html>.
^ permalink raw reply [flat|nested] 25+ messages in thread
* [Bug bootstrap/46528] [4.6 Regression] profiledbootstrap failure
2010-11-17 23:35 [Bug bootstrap/46528] New: [ia64] profiledbootstrap failure steven at gcc dot gnu.org
` (11 preceding siblings ...)
2010-11-24 22:57 ` ubizjak at gmail dot com
@ 2010-11-24 23:06 ` ubizjak at gmail dot com
2010-11-24 23:53 ` ubizjak at gmail dot com
` (9 subsequent siblings)
22 siblings, 0 replies; 25+ messages in thread
From: ubizjak at gmail dot com @ 2010-11-24 23:06 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46528
--- Comment #13 from Uros Bizjak <ubizjak at gmail dot com> 2010-11-24 22:32:15 UTC ---
(In reply to comment #9)
> Soooo ... for those who can reproduce it, can you try with
> -fno-tree-scev-cprop?
Bootstrap also fails with -fno-tree-scev-cprop :(
The configure/build commands were similar to ones in comment 8.
^ permalink raw reply [flat|nested] 25+ messages in thread
* [Bug bootstrap/46528] [4.6 Regression] profiledbootstrap failure
2010-11-17 23:35 [Bug bootstrap/46528] New: [ia64] profiledbootstrap failure steven at gcc dot gnu.org
` (12 preceding siblings ...)
2010-11-24 23:06 ` ubizjak at gmail dot com
@ 2010-11-24 23:53 ` ubizjak at gmail dot com
2010-11-25 0:19 ` ubizjak at gmail dot com
` (8 subsequent siblings)
22 siblings, 0 replies; 25+ messages in thread
From: ubizjak at gmail dot com @ 2010-11-24 23:53 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46528
--- Comment #14 from Uros Bizjak <ubizjak at gmail dot com> 2010-11-24 23:03:30 UTC ---
Created attachment 22520
--> http://gcc.gnu.org/bugzilla/attachment.cgi?id=22520
dump from stageprofile
^ permalink raw reply [flat|nested] 25+ messages in thread
* [Bug bootstrap/46528] [4.6 Regression] profiledbootstrap failure
2010-11-17 23:35 [Bug bootstrap/46528] New: [ia64] profiledbootstrap failure steven at gcc dot gnu.org
` (13 preceding siblings ...)
2010-11-24 23:53 ` ubizjak at gmail dot com
@ 2010-11-25 0:19 ` ubizjak at gmail dot com
2010-11-25 12:41 ` rguenth at gcc dot gnu.org
` (7 subsequent siblings)
22 siblings, 0 replies; 25+ messages in thread
From: ubizjak at gmail dot com @ 2010-11-25 0:19 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46528
--- Comment #15 from Uros Bizjak <ubizjak at gmail dot com> 2010-11-24 23:05:34 UTC ---
Created attachment 22521
--> http://gcc.gnu.org/bugzilla/attachment.cgi?id=22521
stagefedback profile
^ permalink raw reply [flat|nested] 25+ messages in thread
* [Bug bootstrap/46528] [4.6 Regression] profiledbootstrap failure
2010-11-17 23:35 [Bug bootstrap/46528] New: [ia64] profiledbootstrap failure steven at gcc dot gnu.org
` (14 preceding siblings ...)
2010-11-25 0:19 ` ubizjak at gmail dot com
@ 2010-11-25 12:41 ` rguenth at gcc dot gnu.org
2010-11-25 13:40 ` rguenth at gcc dot gnu.org
` (6 subsequent siblings)
22 siblings, 0 replies; 25+ messages in thread
From: rguenth at gcc dot gnu.org @ 2010-11-25 12:41 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46528
--- Comment #16 from Richard Guenther <rguenth at gcc dot gnu.org> 2010-11-25 12:13:14 UTC ---
In your dump files there is a CFG inconsistency that I can't reproduce. At
least basic block numbers are not consistent.
I also see DEBUG insns in stagefeedback but not stageprofile (my dumps
have DEBUG insns in both dump files). Which hints at for example:
<bb 17>:
PROF_edge_counter.65_257 = *.LPBX1[69];
PROF_edge_counter.65_258 = PROF_edge_counter.65_257 + 1;
*.LPBX1[69] = PROF_edge_counter.65_258;
count_61 = macro_real_token_count (macro_39);
<bb 18>:
PROF_edge_counter.65_259 = *.LPBX1[70];
PROF_edge_counter.65_260 = PROF_edge_counter.65_259 + 1;
*.LPBX1[70] = PROF_edge_counter.65_260;
goto <bb 30>;
vs.
<bb 17>:
Invalid sum of outgoing probabilities 0.0%
Invalid sum of outgoing counts 0, should be 211
count_61 = macro_real_token_count (macro_39);
# DEBUG count => count_61
# DEBUG i => 0
goto <bb 29>;
it would be interesting to see when the CFGs start to differ. Can you
also check logs if really -g0 is used at stageprofile and -g at
stagefeedback. I'm trying to look why I have -g consistently.
^ permalink raw reply [flat|nested] 25+ messages in thread
* [Bug bootstrap/46528] [4.6 Regression] profiledbootstrap failure
2010-11-17 23:35 [Bug bootstrap/46528] New: [ia64] profiledbootstrap failure steven at gcc dot gnu.org
` (15 preceding siblings ...)
2010-11-25 12:41 ` rguenth at gcc dot gnu.org
@ 2010-11-25 13:40 ` rguenth at gcc dot gnu.org
2010-11-25 13:48 ` Jan Hubicka
2010-11-25 13:56 ` hubicka at ucw dot cz
` (5 subsequent siblings)
22 siblings, 1 reply; 25+ messages in thread
From: rguenth at gcc dot gnu.org @ 2010-11-25 13:40 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46528
Richard Guenther <rguenth at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |WAITING
--- Comment #17 from Richard Guenther <rguenth at gcc dot gnu.org> 2010-11-25 12:40:32 UTC ---
Please try
Index: gcc/profile.c
===================================================================
--- gcc/profile.c (revision 167127)
+++ gcc/profile.c (working copy)
@@ -940,7 +943,8 @@ branch_prob (void)
for (gsi = gsi_last_bb (bb); !gsi_end_p (gsi); gsi_prev (&gsi))
{
last = gsi_stmt (gsi);
- if (gimple_has_location (last))
+ if (!is_gimple_debug (last)
+ && gimple_has_location (last))
break;
}
we do CFG manipulations based on debug stmts (verified with a later
assert that triggers on -O2 -g -fprofile-use).
It's of course unfortunate that we do this BB splitting at all when
not doing coverage test.
^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [Bug bootstrap/46528] [4.6 Regression] profiledbootstrap failure
2010-11-25 13:40 ` rguenth at gcc dot gnu.org
@ 2010-11-25 13:48 ` Jan Hubicka
0 siblings, 0 replies; 25+ messages in thread
From: Jan Hubicka @ 2010-11-25 13:48 UTC (permalink / raw)
To: rguenth at gcc dot gnu.org; +Cc: gcc-bugs
> we do CFG manipulations based on debug stmts (verified with a later
> assert that triggers on -O2 -g -fprofile-use).
>
> It's of course unfortunate that we do this BB splitting at all when
> not doing coverage test.
The BB splitting is neccesary so the CFG can be solved (i.e. you have explicit edges to exit when program can terminate).
Honza
^ permalink raw reply [flat|nested] 25+ messages in thread
* [Bug bootstrap/46528] [4.6 Regression] profiledbootstrap failure
2010-11-17 23:35 [Bug bootstrap/46528] New: [ia64] profiledbootstrap failure steven at gcc dot gnu.org
` (16 preceding siblings ...)
2010-11-25 13:40 ` rguenth at gcc dot gnu.org
@ 2010-11-25 13:56 ` hubicka at ucw dot cz
2010-11-25 18:09 ` hjl.tools at gmail dot com
` (4 subsequent siblings)
22 siblings, 0 replies; 25+ messages in thread
From: hubicka at ucw dot cz @ 2010-11-25 13:56 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46528
--- Comment #18 from Jan Hubicka <hubicka at ucw dot cz> 2010-11-25 12:56:35 UTC ---
> we do CFG manipulations based on debug stmts (verified with a later
> assert that triggers on -O2 -g -fprofile-use).
>
> It's of course unfortunate that we do this BB splitting at all when
> not doing coverage test.
The BB splitting is neccesary so the CFG can be solved (i.e. you have explicit
edges to exit when program can terminate).
Honza
^ permalink raw reply [flat|nested] 25+ messages in thread
* [Bug bootstrap/46528] [4.6 Regression] profiledbootstrap failure
2010-11-17 23:35 [Bug bootstrap/46528] New: [ia64] profiledbootstrap failure steven at gcc dot gnu.org
` (17 preceding siblings ...)
2010-11-25 13:56 ` hubicka at ucw dot cz
@ 2010-11-25 18:09 ` hjl.tools at gmail dot com
2010-11-25 19:59 ` ubizjak at gmail dot com
` (3 subsequent siblings)
22 siblings, 0 replies; 25+ messages in thread
From: hjl.tools at gmail dot com @ 2010-11-25 18:09 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46528
--- Comment #19 from H.J. Lu <hjl.tools at gmail dot com> 2010-11-25 17:43:35 UTC ---
I started profiledbootstrap tracking:
http://gcc.gnu.org/ml/gcc-regression/2010-11/msg00378.html
^ permalink raw reply [flat|nested] 25+ messages in thread
* [Bug bootstrap/46528] [4.6 Regression] profiledbootstrap failure
2010-11-17 23:35 [Bug bootstrap/46528] New: [ia64] profiledbootstrap failure steven at gcc dot gnu.org
` (18 preceding siblings ...)
2010-11-25 18:09 ` hjl.tools at gmail dot com
@ 2010-11-25 19:59 ` ubizjak at gmail dot com
2010-11-26 9:54 ` rguenth at gcc dot gnu.org
` (2 subsequent siblings)
22 siblings, 0 replies; 25+ messages in thread
From: ubizjak at gmail dot com @ 2010-11-25 19:59 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46528
Uros Bizjak <ubizjak at gmail dot com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|WAITING |NEW
--- Comment #20 from Uros Bizjak <ubizjak at gmail dot com> 2010-11-25 19:34:03 UTC ---
(In reply to comment #17)
> Please try
>
> Index: gcc/profile.c
> ===================================================================
> --- gcc/profile.c (revision 167127)
> +++ gcc/profile.c (working copy)
> @@ -940,7 +943,8 @@ branch_prob (void)
> for (gsi = gsi_last_bb (bb); !gsi_end_p (gsi); gsi_prev (&gsi))
> {
> last = gsi_stmt (gsi);
> - if (gimple_has_location (last))
> + if (!is_gimple_debug (last)
> + && gimple_has_location (last))
> break;
> }
Great, this bootstrapped OK for -enable-languages=c.
^ permalink raw reply [flat|nested] 25+ messages in thread
* [Bug bootstrap/46528] [4.6 Regression] profiledbootstrap failure
2010-11-17 23:35 [Bug bootstrap/46528] New: [ia64] profiledbootstrap failure steven at gcc dot gnu.org
` (19 preceding siblings ...)
2010-11-25 19:59 ` ubizjak at gmail dot com
@ 2010-11-26 9:54 ` rguenth at gcc dot gnu.org
2010-11-26 9:55 ` rguenth at gcc dot gnu.org
2010-12-06 19:31 ` jakub at gcc dot gnu.org
22 siblings, 0 replies; 25+ messages in thread
From: rguenth at gcc dot gnu.org @ 2010-11-26 9:54 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46528
--- Comment #21 from Richard Guenther <rguenth at gcc dot gnu.org> 2010-11-26 09:50:06 UTC ---
Author: rguenth
Date: Fri Nov 26 09:49:47 2010
New Revision: 167171
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=167171
Log:
2010-11-26 Richard Guenther <rguenther@suse.de>
PR tree-optimization/46528
PR debug/46338
* profile.c (branch_prob): Do not split blocks based on locations
from debug statements.
Modified:
trunk/gcc/ChangeLog
trunk/gcc/profile.c
^ permalink raw reply [flat|nested] 25+ messages in thread
* [Bug bootstrap/46528] [4.6 Regression] profiledbootstrap failure
2010-11-17 23:35 [Bug bootstrap/46528] New: [ia64] profiledbootstrap failure steven at gcc dot gnu.org
` (20 preceding siblings ...)
2010-11-26 9:54 ` rguenth at gcc dot gnu.org
@ 2010-11-26 9:55 ` rguenth at gcc dot gnu.org
2010-12-06 19:31 ` jakub at gcc dot gnu.org
22 siblings, 0 replies; 25+ messages in thread
From: rguenth at gcc dot gnu.org @ 2010-11-26 9:55 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46528
Richard Guenther <rguenth at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution| |FIXED
--- Comment #22 from Richard Guenther <rguenth at gcc dot gnu.org> 2010-11-26 09:51:50 UTC ---
Fixed.
^ permalink raw reply [flat|nested] 25+ messages in thread
* [Bug bootstrap/46528] [4.6 Regression] profiledbootstrap failure
2010-11-17 23:35 [Bug bootstrap/46528] New: [ia64] profiledbootstrap failure steven at gcc dot gnu.org
` (21 preceding siblings ...)
2010-11-26 9:55 ` rguenth at gcc dot gnu.org
@ 2010-12-06 19:31 ` jakub at gcc dot gnu.org
22 siblings, 0 replies; 25+ messages in thread
From: jakub at gcc dot gnu.org @ 2010-12-06 19:31 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46528
--- Comment #23 from Jakub Jelinek <jakub at gcc dot gnu.org> 2010-12-06 19:30:39 UTC ---
Author: jakub
Date: Mon Dec 6 19:30:31 2010
New Revision: 167515
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=167515
Log:
PR tree-optimization/46528
PR debug/46338
* profile.c (branch_prob): Make sure last is never set to a debug
stmt.
Modified:
trunk/gcc/ChangeLog
trunk/gcc/profile.c
^ permalink raw reply [flat|nested] 25+ messages in thread