Skip to content

net-libs/nodejs: filter out -flto from *FLAGS; -Ofast workarounds#779

Merged
InBetweenNames merged 4 commits into
InBetweenNames:masterfrom
jelinekto:master
Sep 26, 2021
Merged

net-libs/nodejs: filter out -flto from *FLAGS; -Ofast workarounds#779
InBetweenNames merged 4 commits into
InBetweenNames:masterfrom
jelinekto:master

Conversation

@jelinekto
Copy link
Copy Markdown
Contributor

net-libs/nodejs: There's lto USE flag that should be used to inject -flto instead, also LTO currently does not work with GCC 11 (you'll be asked to disable the USE flag when you try to emerge it).

app-editors/emacs: You'll be explicitly asked to disable -ffinite-math-only during emerge.

gui-apps/gammastep: Fork of x11-misc/redshift, -ffinite-math-only breaks it in the exactly same way.

Tomáš Jelínek added 4 commits August 20, 2021 11:12
One should use the USE flag to enable LTO instead (which does not work with GCC 11 as of now).
Ebuild explicitly asks you to.
Builds but becomes basically useless, i.e. just throws out an error about an invalid brightness value.
@jelinekto
Copy link
Copy Markdown
Contributor Author

I've added a (possibly) temporary workaround for Firefox; -Ofast and -ffast-math get filtered by flagomatic and specifying -fno-honor-infinities -fno-honor-nans -fno-math-errno -ffinite-math-only -fassociative-math -freciprocal-math -fno-signed-zeros -fno-trapping-math -ffp-contract=fast (aka the flags activated by Clang -ffast-math) causes the build to fail.

Obviously investigating Firefox build failures without a CPU that is beefy enough is an absolute bitch and would cause me to lose my will to live pretty fast, so I suggest falling back to -fno-trapping-math -fno-math-errno which makes it possible to at least build the package with both Clang and GCC for now. So far I can tell you removing -ffinite-math-only doesn't help.

@InBetweenNames InBetweenNames merged commit 4c0b4b6 into InBetweenNames:master Sep 26, 2021
@InBetweenNames
Copy link
Copy Markdown
Owner

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants