Skip to content
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

Meros uses -z muldefs which is inherently risky. #310

Open
kayabaNerve opened this issue May 17, 2021 · 0 comments
Open

Meros uses -z muldefs which is inherently risky. #310

kayabaNerve opened this issue May 17, 2021 · 0 comments
Labels
bug Something isn't working optimization This could be better

Comments

@kayabaNerve
Copy link
Member

It's required at this time due to:

  1. Having multiple Rust libraries.
  2. https://github.com/rust-lang/rust/issues/ 44322

I did try disabling LTO in our second Rust library before it was merged, yet that alone didn't work. This should be improved though. Labelled as a bug due to the risk and optimization as it is an improvement.

@kayabaNerve kayabaNerve added bug Something isn't working optimization This could be better labels May 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working optimization This could be better
Development

No branches or pull requests

1 participant