-
Notifications
You must be signed in to change notification settings - Fork 96
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
media-libs/gst-plugins-good: disable LTO #774
base: master
Are you sure you want to change the base?
Conversation
Builds fine with |
Did not work for me with |
Any update on this one? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@aruhier Sorry, I didn't see your PR and made a duplicate. I'm closing in favor of your PR. However, I would suggest you change media-libs/gst-plugins-good
to =media-libs/gst-plugins-good-1.18.4
as media-libs/gst-plugins-good-1.16.3
builds just fine with LTO.
12ea170
to
aaefccc
Compare
Fails to compile.
aaefccc
to
ac37e0a
Compare
@ran-dall: no worries, and thanks for your tests! |
didn't work with |
affects media-libs/gst-plugins-good-1.20.1 as well. Please merge. |
A fix for this has been merged to meson 3 days ago. As soon as they make a new release with that fix and meson gets updated in Gentoo this issue should disappear. Since the issue is really in another package and not gst-plugins-good if this gets merged it's something that should also get reverted once a fixed meson enters either the gentoo unstable or stable branch...? It seems like all gst-plugins-good versions >1.16.3 will need the fix in Meson in order to build with LTO. |
Sounds good, once I can check that it compiles with an updated meson, I'll close this PR. |
I just tested myself that the current live build of meson in Gentoo does allow newer versions of gst-plugins-good to build with LTO. So hopefully it won't be long now until an ~amd64 version becomes available. |
Meson 0.62.1 was released on the 23rd of April and has also been in the unstable Gentoo tree for quite a while. Unfortunately though this fix was not included there so gst-plugins-good versions > 1.16.3 still fails to build using the Gentoo unstable Meson. It probably works for some architectures but most of us are running amd64 and there it's broken. This bug is triggered on certain architectures with older versions of Meson if you try to build with certain debug or LTO flags and the source code includes raw assembly code which is what was added to gst-plugins-good > 1.16.3. |
Meson 0.62.2 has been released but it still fails to build gst-plugins-good. Latest meson-9999 still builds it without any issues. |
meson-0.63.0 has been released to gentoo unstable and it successfully builds gst-plugins-good for me with LTO enabled |
Fails to compile.
Compiling with LTO gives the following error:
I tried to disable other optimizations, such as IPA-PTA, forcing to use ld-gold, but it didn't solve the issue.