[Bug lto/87988] [9 regression] Streaming of ABSTRACT_ORIGIN is expensive

2018-12-21 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87988 Richard Biener changed: What|Removed |Added Keywords||compile-time-hog, deferred,

[Bug lto/87988] [9 regression] Streaming of ABSTRACT_ORIGIN is expensive

2018-11-29 Thread rguenther at suse dot de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87988 --- Comment #10 from rguenther at suse dot de --- On Wed, 28 Nov 2018, hubicka at gcc dot gnu.org wrote: > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87988 > > --- Comment #9 from Jan Hubicka --- > We still have: >/* When not generating

[Bug lto/87988] [9 regression] Streaming of ABSTRACT_ORIGIN is expensive

2018-11-28 Thread hubicka at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87988 --- Comment #9 from Jan Hubicka --- We still have: /* When not generating debug info we can eliminate info on unused variables. */ else if (!flag_auto_profile &&

[Bug lto/87988] [9 regression] Streaming of ABSTRACT_ORIGIN is expensive

2018-11-27 Thread rguenther at suse dot de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87988 --- Comment #8 from rguenther at suse dot de --- On November 27, 2018 12:01:03 PM GMT+01:00, "hubicka at gcc dot gnu.org" wrote: >https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87988 > >--- Comment #7 from Jan Hubicka --- >Hi, >ltrans files are 1

[Bug lto/87988] [9 regression] Streaming of ABSTRACT_ORIGIN is expensive

2018-11-27 Thread hubicka at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87988 --- Comment #7 from Jan Hubicka --- Hi, ltrans files are 1374K without and 1339K with patch. WPA report without patch: [WPA] read 13690507 SCCs of average size 1.397311 [WPA] 19129895 tree bodies read in total [WPA] tree SCC table: size 4194301,

[Bug lto/87988] [9 regression] Streaming of ABSTRACT_ORIGIN is expensive

2018-11-26 Thread hubicka at ucw dot cz
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87988 --- Comment #6 from Jan Hubicka --- > > Honza - can you test the effect of this patch please? Thanks! I am just redoing the tests (rebuilding firefoxes with updated tree), so i will do that today or tomorrow. Honza

[Bug lto/87988] [9 regression] Streaming of ABSTRACT_ORIGIN is expensive

2018-11-26 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87988 --- Comment #5 from Richard Biener --- Created attachment 45092 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=45092&action=edit better patch This is a less hacky approach mimicking TREE_DIEs. It elides BLOCK BLOCK_ABSTRACT_ORIGIN if that

[Bug lto/87988] [9 regression] Streaming of ABSTRACT_ORIGIN is expensive

2018-11-13 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87988 --- Comment #4 from Richard Biener --- OK, I'd rather not put this hack on trunk. Iff then a very limited scope TREE_DIE (same cases as the hack) should be brought in.

[Bug lto/87988] [9 regression] Streaming of ABSTRACT_ORIGIN is expensive

2018-11-13 Thread hubicka at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87988 --- Comment #3 from Jan Hubicka --- Thanks, I will give it a try. Note that the numbers I posted are from trunk and trunk with abstract origin streaming disabled. Thanks to the type streaming reorg we do not have overall regression relative to

[Bug lto/87988] [9 regression] Streaming of ABSTRACT_ORIGIN is expensive

2018-11-13 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87988 --- Comment #2 from Richard Biener --- Created attachment 44995 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=44995&action=edit untested patch It's surprisingly difficult to hack around things ... but the attached at least survives lto.ex

[Bug lto/87988] [9 regression] Streaming of ABSTRACT_ORIGIN is expensive

2018-11-13 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87988 Richard Biener changed: What|Removed |Added Status|UNCONFIRMED |ASSIGNED Last reconfirmed|