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

enable LTO for riscv #453

Closed
nickdesaulniers opened this issue Oct 5, 2022 · 8 comments · Fixed by #690
Closed

enable LTO for riscv #453

nickdesaulniers opened this issue Oct 5, 2022 · 8 comments · Fixed by #690

Comments

@nickdesaulniers
Copy link
Member

I don't see it pushed to the riscv tree yet, but LTO for riscv is coming.

@nickdesaulniers nickdesaulniers added the blocked (linux) Blocked on an issue with Linux upstream label Oct 5, 2022
@nickdesaulniers
Copy link
Member Author

@nathanchance
Copy link
Member

I don't see it on any branch in https://git.kernel.org/pub/scm/linux/kernel/git/riscv/linux.git/ nor in mainline.

@nickdesaulniers
Copy link
Member Author

cc @twd2

@twd2
Copy link
Member

twd2 commented Oct 21, 2022

I saw my patch was once applied to for-next but later that commit was removed from the tree.
Is there anything I can improve? @palmer-dabbelt

@nathanchance
Copy link
Member

@twd2 I sent a ping for it recently but I think it might just be better to resend the patch at this point. The RISC-V folks recently went back to tracking patches via Patchwork, which should help ensure the patch does not get lost (as well as make it obvious to you when it has been picked up).

@nathanchance
Copy link
Member

I sent v3 due to arch/riscv/kernel/pi needing LTO disabled: https://lore.kernel.org/20231003-riscv-lto-v3-1-8aca61a4ecb4@kernel.org/

@twd2
Copy link
Member

twd2 commented Oct 4, 2023

Cool! Thanks!

@nathanchance
Copy link
Member

This has been accepted and is now available in -next, so we should be able to get this wired up there:

https://git.kernel.org/next/linux-next/c/021d23428bdbae032294e8f4a29cb53cb50ae71c

@nathanchance nathanchance removed the blocked (linux) Blocked on an issue with Linux upstream label Jan 24, 2024
@nathanchance nathanchance linked a pull request Jan 25, 2024 that will close this issue
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 a pull request may close this issue.

3 participants