[Bug c++/77620] Generic compile time regression of 7.0

2017-02-01 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77620

--- Comment #8 from Jonathan Wakely  ---
We get reports like this every few months, and nobody ever uses -ftime-report
before filing a bug. I think something in the -v output would be useful.

[Bug c++/77620] Generic compile time regression of 7.0

2017-02-01 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77620

--- Comment #7 from Andrew Pinski  ---
We already output one if you use -ftime-report.

[Bug c++/77620] Generic compile time regression of 7.0

2017-02-01 Thread petschy at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77620

--- Comment #6 from petschy at gmail dot com ---
Would it be sensible to put an extra line to the output of 'gcc/g++ -v' if the
slow checks are enabled, which just states this fact / warns about (possibly
mentioning the use of --enable-checking=release at configure)? Future tickets
like this might be avoided this way.

[Bug c++/77620] Generic compile time regression of 7.0

2016-09-16 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77620

--- Comment #5 from Andrew Pinski  ---
(In reply to petschy from comment #2)
> Thanks, --enable-checking=release did the trick. Those unreleased checks
> definitely have some runtime cost :)

Yes in some cases there are O(n^2) checking functions which hurt compile time. 
Plus running more and more checking hurts cache locality too.

[Bug c++/77620] Generic compile time regression of 7.0

2016-09-16 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77620

--- Comment #3 from Andrew Pinski  ---
Closing as invalid then.  That is --enable-checking=release gets us to where
6.2.1 was.

[Bug c++/77620] Generic compile time regression of 7.0

2016-09-16 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77620

Andrew Pinski  changed:

   What|Removed |Added

 Status|WAITING |RESOLVED
 Resolution|--- |INVALID

--- Comment #4 from Andrew Pinski  ---
.

[Bug c++/77620] Generic compile time regression of 7.0

2016-09-16 Thread petschy at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77620

--- Comment #2 from petschy at gmail dot com ---
Thanks, --enable-checking=release did the trick. Those unreleased checks
definitely have some runtime cost :)

My project was built in 3m35 with -O3, and the gcc master branch:

 7.0.0
-O0  7m30
-O1  7m00
-O2  8m23
-O3  9m02

These are effectively the same as with 6.2.1.

Sorry for the noise.

[Bug c++/77620] Generic compile time regression of 7.0

2016-09-16 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77620

Andrew Pinski  changed:

   What|Removed |Added

 Status|UNCONFIRMED |WAITING
   Last reconfirmed||2016-09-16
 Ever confirmed|0   |1

--- Comment #1 from Andrew Pinski  ---
What happens if you configure gcc trunk with --enable-checking=release ?  The
released versions default to that already and when GCC 7 is released, it will
also be that default too.