please consider to add lto useflag to app-emulation/dxvk #10

Closed
opened 2020-10-23 09:27:59 +02:00 by stefson · 4 comments

recently I learned about link time optimization. Do you think it's possible to add a useflag to the dxvk ebuild? This way it might be possible to squeeze out a bit more performance at runtime.

sources:

https://bugs.gentoo.org/745372#c13

recently I learned about link time optimization. Do you think it's possible to add a useflag to the dxvk ebuild? This way it might be possible to squeeze out a bit more performance at runtime. sources: https://bugs.gentoo.org/745372#c13
Owner

I think it would make more sense if you add the flag to your make.conf or /etc/portage/package.env?

I'm reluctant to directly support it because upstream doesn't use it and has no option in its build system for it.

If you write something like CXXFLAGS="${CXXFLAGS} -flto" to /etc/portage/env/use_flto.conf and app-emulation/dxvk use_flto.conf to /etc/portage/package.env/dxvk it should work.

I think it would make more sense if you add the flag to your make.conf or [`/etc/portage/package.env`](https://wiki.gentoo.org/wiki//etc/portage/package.env)? I'm reluctant to directly support it because [upstream doesn't use it](https://github.com/doitsujin/dxvk/issues/1782#issuecomment-701280398) and has no option in its build system for it. If you write something like `CXXFLAGS="${CXXFLAGS} -flto"` to `/etc/portage/env/use_flto.conf` and `app-emulation/dxvk use_flto.conf` to `/etc/portage/package.env/dxvk` it should work.
Author

uhm, ok, I thought the use of lto is so widespread that upstream supports it out of the box.

uhm, ok, I thought the use of lto is so widespread that upstream supports it out of the box.
Owner

app-emulation/dxvk is in the main repo now and masked for removal here. LTO seems to work fine: https://bugs.gentoo.org/854516.

app-emulation/dxvk is in the main repo now and masked for removal here. LTO seems to work fine: <https://bugs.gentoo.org/854516>.
Author

yes indeed, lto works presumably with the new dev-util/ming64-toolchain package

yes indeed, lto works presumably with the new ` dev-util/ming64-toolchain` package
Sign in to join this conversation.
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: tastytea/overlay#10
No description provided.