[Bug lto/50165] [4.7 Regression] Huge build time regression (Firefox lto build)

2011-08-26 Thread markus at trippelsdorf dot de
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50165 Markus Trippelsdorf changed: What|Removed |Added Status|NEW |RESOLVED Resolution|

[Bug lto/50165] [4.7 Regression] Huge build time regression (Firefox lto build)

2011-08-26 Thread matz at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50165 --- Comment #11 from Michael Matz 2011-08-26 16:02:21 UTC --- Author: matz Date: Fri Aug 26 16:02:17 2011 New Revision: 178118 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=178118 Log: PR lto/50165 * lto-streamer-in.c (canon_file

[Bug lto/50165] [4.7 Regression] Huge build time regression (Firefox lto build)

2011-08-26 Thread markus at trippelsdorf dot de
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50165 --- Comment #10 from Markus Trippelsdorf 2011-08-26 13:25:33 UTC --- (In reply to comment #8) > Try this: > > Index: lto-streamer-in.c > === > --- lto-streamer-in.c (revision 1780

[Bug lto/50165] [4.7 Regression] Huge build time regression (Firefox lto build)

2011-08-26 Thread dnovillo at google dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50165 --- Comment #9 from dnovillo at google dot com 2011-08-26 12:21:33 UTC --- I will be with limited e-mail access until 7-Sep-2011. I will read your message when I get back. Diego.

[Bug lto/50165] [4.7 Regression] Huge build time regression (Firefox lto build)

2011-08-26 Thread matz at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50165 Michael Matz changed: What|Removed |Added CC||matz at gcc dot gnu.org --- Comment #8 fro

[Bug lto/50165] [4.7 Regression] Huge build time regression (Firefox lto build)

2011-08-24 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50165 --- Comment #7 from Richard Guenther 2011-08-24 10:46:17 UTC --- (In reply to comment #6) > (In reply to comment #4) > > > - > > > - if (ds1->len == ds2->len) > > > -return memcmp (ds1->s, ds2->s, ds1->len) == 0; > > > - > > > - return 0; >

[Bug lto/50165] [4.7 Regression] Huge build time regression (Firefox lto build)

2011-08-24 Thread markus at trippelsdorf dot de
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50165 --- Comment #6 from Markus Trippelsdorf 2011-08-24 10:41:15 UTC --- (In reply to comment #4) > > - > > - if (ds1->len == ds2->len) > > -return memcmp (ds1->s, ds2->s, ds1->len) == 0; > > - > > - return 0; > > + return strcmp (ds1->s, ds2->

[Bug lto/50165] [4.7 Regression] Huge build time regression (Firefox lto build)

2011-08-24 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50165 --- Comment #5 from Richard Guenther 2011-08-24 10:37:18 UTC --- Btw, as ->len includes the trailing zero, the new hash function hashes in a zero and the memcmp compares one byte unnecessarily. To make the transform 1:1 it would be static inlin

[Bug lto/50165] [4.7 Regression] Huge build time regression (Firefox lto build)

2011-08-24 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50165 --- Comment #4 from Richard Guenther 2011-08-24 10:16:56 UTC --- (In reply to comment #3) > The following patch fixes the problem for me: > > diff --git a/gcc/data-streamer.h b/gcc/data-streamer.h > index c413a75..acf1305 100644 > --- a/gcc/data

[Bug lto/50165] [4.7 Regression] Huge build time regression (Firefox lto build)

2011-08-24 Thread markus at trippelsdorf dot de
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50165 --- Comment #3 from Markus Trippelsdorf 2011-08-24 09:53:24 UTC --- The following patch fixes the problem for me: diff --git a/gcc/data-streamer.h b/gcc/data-streamer.h index c413a75..acf1305 100644 --- a/gcc/data-streamer.h +++ b/gcc/data-strea

[Bug lto/50165] [4.7 Regression] Huge build time regression (Firefox lto build)

2011-08-24 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50165 Richard Guenther changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|

[Bug lto/50165] [4.7 regression] Huge build time regression (Firefox lto build)

2011-08-23 Thread markus at trippelsdorf dot de
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50165 --- Comment #1 from Markus Trippelsdorf 2011-08-23 16:38:22 UTC --- This regression is caused by revision 177571. Reverting only 177661 and 177704 doesn't solve the problem. Reverting all three revisions (77571, 177661 and 177704) solves the prob