Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 920092

Summary: app-doc/doxygen ebuild should filter out -fipa-pta
Product: Gentoo Linux Reporter: David Jones <mhblruzy>
Component: Current packagesAssignee: Matthias Maier <tamiko>
Status: RESOLVED FIXED    
Severity: normal    
Priority: Normal    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---

Description David Jones 2023-12-16 02:11:07 UTC
Due to the way the code is written this optimization causes the compiler to waster a lot of time on useless optimizations.
Comment 1 Sam James archtester Gentoo Infrastructure gentoo-dev Security 2023-12-16 02:14:13 UTC
Yep.

Note that the gcc upstream recommendation is to only bother using -fipa-pta (as it is currently implemented, anyway) on packages where it's been benchmarked to be worth it, given the compile time impact is so dreadful.

Anyway, I'll add that in as it's cheap, just keep that in mind.
Comment 2 Larry the Git Cow gentoo-dev 2023-12-16 03:08:53 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=68ed17b3a3185390ee87d59080ab7988cf038a93

commit 68ed17b3a3185390ee87d59080ab7988cf038a93
Author:     Sam James <sam@gentoo.org>
AuthorDate: 2023-12-16 03:08:00 +0000
Commit:     Sam James <sam@gentoo.org>
CommitDate: 2023-12-16 03:08:10 +0000

    app-doc/doxygen: filter -fipa-pta
    
    Very slow to compile and not worth it here.
    
    Closes: https://bugs.gentoo.org/920092
    Signed-off-by: Sam James <sam@gentoo.org>

 app-doc/doxygen/doxygen-1.9.8.ebuild | 2 ++
 1 file changed, 2 insertions(+)

Additionally, it has been referenced in the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=712f6dc0926c54896d6c0717b2c8ccc4d38af2d4

commit 712f6dc0926c54896d6c0717b2c8ccc4d38af2d4
Author:     Sam James <sam@gentoo.org>
AuthorDate: 2023-12-16 03:07:01 +0000
Commit:     Sam James <sam@gentoo.org>
CommitDate: 2023-12-16 03:08:09 +0000

    app-doc/doxygen: improve -O2 forcing
    
    No idea why this is there but improve the output (less noisy) and add a
    TODO comment.
    
    Bug: https://bugs.gentoo.org/920092
    Signed-off-by: Sam James <sam@gentoo.org>

 app-doc/doxygen/doxygen-1.9.8.ebuild | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)