[WIP] Add base definitions for riscv64 + musl

[ddevault]

No description provided.

[rust-highfive]

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @JohnTitor (or someone else) soon.

If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.

Please see the contribution instructions for more information.

[JohnTitor]

Is there any riscv target that uses musl as target_env? Also, it'd be great if you could provide some context.

[ddevault]

There will be soon. I am working on porting Alpine Linux to RISC-V. You can follow along with that work here:

https://gitlab.alpinelinux.org/alpine/aports/-/issues/12167

[JohnTitor]

Then I'd love to see a target spec as JSON and run CI for it, to make sure the build. And you should consider sending a PR to add a new target to rust-lang/rust, I think.

[ddevault]

Yeah, I will have something for rust-lang/rust as well. I need to finish bootstrapping the damn thing first, what a miserable process. Can you elaborate on the target spec as JSON issue?

[JohnTitor]

Can you elaborate on the target spec as JSON issue?

We can pass a target spec as JSON, like https://github.com/rust-lang/libc/blob/master/ci/switch.json, to run the build test.

[jyn514]

Yeah, I will have something for rust-lang/rust as well. I need to finish bootstrapping the damn thing first, what a miserable process. Can you elaborate on the target spec as JSON issue?

It might be helpful to see how aarch64 did it: rust-lang/rust#74541

[tesuji]

riscv{32,64}gc-unknown-linux-musl has been added in rust-lang/rust#82202

[bors]

☔ The latest upstream changes (presumably #2071) made this pull request unmergeable. Please resolve the merge conflicts.

[JohnTitor]

riscv{32,64}gc-unknown-linux-musl has been added in rust-lang/rust#82202

Great! I'm not sure how difficult to run ctest on that target but at least we should run the build check on CI, as I said before. It can be done easily by tweaking here: https://github.com/rust-lang/libc/blob/master/ci/build.sh

[JohnTitor]

I'm going to close as inactive, feel free to rebase and re-open when you get some time :)