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

gold linker from binutils #55

Open
abrisco opened this issue Jan 10, 2023 · 6 comments
Open

gold linker from binutils #55

abrisco opened this issue Jan 10, 2023 · 6 comments

Comments

@abrisco
Copy link

abrisco commented Jan 10, 2023

I'm not sure if this is the right place to ask but would it be possible to start including the gold linker in artifacts?

@wdlkmpx
Copy link

wdlkmpx commented Jan 15, 2023

The configure opt is --enable-gold I think

They say it was added by google and has been in maintenance mode for years

@abrisco
Copy link
Author

abrisco commented Jan 17, 2023

The configure option to include it in artifacts? Is there a pull request I could make to enable that?

@wdlkmpx
Copy link

wdlkmpx commented Jan 17, 2023

You have to wait until the next release

If you want directly edit buildroot
https://github.com/buildroot/buildroot/tree/master/package/binutils

But I guess something like this can be added to the local config in this repo
BR2_BINUTILS_EXTRA_CONFIG_OPTIONS = --enable-gold

@wdlkmpx
Copy link

wdlkmpx commented Jan 18, 2023

I agree the gold linker should be included in the tarballs

@abrisco
Copy link
Author

abrisco commented Jan 18, 2023

It's just not clear to me how to accomplish that. Happy to help if I can.

@wdlkmpx
Copy link

wdlkmpx commented Jan 26, 2023

Perhaps adding this line: BR2_BINUTILS_EXTRA_CONFIG_OPTIONS=--enable-gold
to this file
https://github.com/bootlin/toolchains-builder/blob/93f88ed30a4fd13ada7e2debf0d035855a690585/configs/optionals/*.config

I'm not sure, but you can build it yourself (./build.sh as root) and see if the gold linker appears, this project builds for many architectures, so basically that change has to be tested in all supported architectures I guess...

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

No branches or pull requests

2 participants