https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81033
--- Comment #33 from Richard Biener ---
The summary is now misleading as well. IMHO the bug shouldn't have been
overloaded with the fallout of the original fix.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81033
Eric Gallager changed:
What|Removed |Added
Priority|P1 |P2
--- Comment #32 from Eric Gallager
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81033
--- Comment #31 from Dominique d'Humieres ---
> Any progress on this?
> Or shall we just disable completely hot/cold function partitioning
> on darwin till then?
IMO it would be better to set the priority to P2.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81033
Jakub Jelinek changed:
What|Removed |Added
CC||jakub at gcc dot gnu.org
--- Comment
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81033
--- Comment #29 from Iain Sandoe ---
(In reply to Dominique d'Humieres from comment #28)
> Bootstrap is fixed, but the fix did not please to Iain Sandoe.
The fix allows bootstrap to proceed, but doesn't solve the underlying problem
(which is
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81033
--- Comment #28 from Dominique d'Humieres ---
Bootstrap is fixed, but the fix did not please to Iain Sandoe.
Dominique
> Le 19 nov. 2017 à 17:40, hubicka at gcc dot gnu.org
> a écrit :
>
>
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81033
--- Comment #27 from Jan Hubicka ---
Is this fixed now?
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81033
Eric Gallager changed:
What|Removed |Added
Status|REOPENED|ASSIGNED
CC|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81033
--- Comment #25 from Dominique d'Humieres ---
I have posted the results on darwin for r250610 at
https://gcc.gnu.org/ml/gcc-testresults/2017-07/msg02582.html. I see 646
failures compared to 318 with r249104.
I am planning to open new Pos for
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81033
--- Comment #24 from Jan Hubicka ---
I think the patch still need to be updated to correctly handle the symbols
whose names are parethetised which was mentioned on the IRC (weird thing that
seems to be used in objC++ API). That should fix good
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81033
Dominique d'Humieres changed:
What|Removed |Added
Status|RESOLVED|REOPENED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81033
Andrew Pinski changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81033
--- Comment #21 from simon at pushface dot org ---
Actually, r249930 was the last related revision (cleans up spaces).
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81033
--- Comment #20 from simon at pushface dot org ---
r249926
See https://gcc.gnu.org/ml/gcc-patches/2017-07/msg00121.html
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81033
--- Comment #19 from d25fe0be@ ---
Bootstrap seems to work for me now.
Not sure which revision brings it back to normal though.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81033
--- Comment #18 from Eric Gallager ---
(In reply to Dominique d'Humieres from comment #10)
> The last patch in comment 8 exposes another problem: buf[128] is too small.
>
> The following patch which uses the trick proposed in IRC by Richi to
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81033
--- Comment #17 from simon at pushface dot org ---
I applied your patches (comment 12, comment 13) to the gcc-8-20170702 snapshot.
Building on darwin 16.6.0 with gcc-7.1.0 as the stage1 compiler was successful
(languages=c,c++,ada).
make
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81033
--- Comment #16 from Dominique d'Humieres ---
Created attachment 41677
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=41677=edit
assembly with the fixes before r249930
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81033
--- Comment #15 from Dominique d'Humieres ---
Created attachment 41676
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=41676=edit
Assembly with the patch for r249930
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81033
--- Comment #14 from Dominique d'Humieres ---
Revision r249930 breaks some objc tests:
% /opt/gcc/gcc8p-249834p3/bin/g++
/opt/gcc/work/gcc/testsuite/objc.dg/torture/forward-1.m
-L/opt/gcc/build_w/x86_64-apple-darwin16.6.0/libobjc/.libs -lobjc
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81033
--- Comment #13 from dominiq at gcc dot gnu.org ---
Author: dominiq
Date: Mon Jul 3 17:42:54 2017
New Revision: 249930
URL: https://gcc.gnu.org/viewcvs?rev=249930=gcc=rev
Log:
2017-07-03 Dominique d'Humieres
PR
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81033
--- Comment #12 from dominiq at gcc dot gnu.org ---
Author: dominiq
Date: Mon Jul 3 15:53:56 2017
New Revision: 249926
URL: https://gcc.gnu.org/viewcvs?rev=249926=gcc=rev
Log:
2017-07-03 Dominique d'Humieres
PR
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81033
simon at pushface dot org changed:
What|Removed |Added
CC||simon at pushface dot org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81033
--- Comment #10 from Dominique d'Humieres ---
The last patch in comment 8 exposes another problem: buf[128] is too small.
The following patch which uses the trick proposed in IRC by Richi to split
assemble_name_raw seems to work
---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81033
--- Comment #9 from Eric Gallager ---
(In reply to Jan Hubicka from comment #8)
>
>
> Looking into the error I suppose problem is:
> ___cold_sect_of_allocate:
> __ZN9__gnu_cxx16bitmap_allocatorIcE8allocateEmPKv.cold.42:
>
> and bit later
>
>
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81033
--- Comment #8 from Jan Hubicka ---
We discussed this on IRC some time ago. The problem is that bb partitioning
seems broken on Darwin. Until the mentioned revision only -fprofile-use enabled
it and thus the bug probably went unnoticed because
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81033
--- Comment #7 from Eric Gallager ---
For me the error is:
/private/var/root/gcc-git/my_oddly_named_builddir/./prev-gcc/xg++
-B/private/var/root/gcc-git/my_oddly_named_builddir/./prev-gcc/
-B/usr/local/i386-apple-darwin9.8.0/bin/ -nostdinc++
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81033
--- Comment #6 from Rainer Orth ---
Jan,
can you *please* have a look at this bug? Darwin bootstrap is broken for
almost
a week with no signs of activity to fix this!
Thanks.
Rainer
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81033
--- Comment #5 from Dominique d'Humieres ---
Revision r249105 breaks bootstrap again, even with the patch in comment 4.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81033
Dominique d'Humieres changed:
What|Removed |Added
Summary|[8 Regression] Bootstrap|[8 Regression] Revision
30 matches
Mail list logo