Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 920092 - app-doc/doxygen ebuild should filter out -fipa-pta
Summary: app-doc/doxygen ebuild should filter out -fipa-pta
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Matthias Maier
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-12-16 02:11 UTC by David Jones
Modified: 2024-03-22 08:46 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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(-)