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

libc: changes to ppc64le musl branch to support building of rust on A… #1023

Merged
merged 1 commit into from
Jun 28, 2018

Conversation

mksully22
Copy link
Contributor

…lpine

This PR includes changes to the libc musl branch to include the correct defines & declarations to support powerpc64. Values that needed changes to a definition for powerpc64.rs that existed higher in the branch also resulted in a change that moved the definition down to the b32/mod.rs, b64/x86_64.rs to ensure that builds continued to work on those architectures.

Verification was done building rust for both ppc64le and x86_64 on Alpine as described in the git project
https://github.com/mksully22/ppc64le_alpine_rust_1.26.2

@rust-highfive
Copy link

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @alexcrichton (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.

…lpine

amend to add style changes
amend to add style changes
@alexcrichton
Copy link
Member

@bors: r+

Thanks!

@bors
Copy link
Contributor

bors commented Jun 27, 2018

📌 Commit dcff154 has been approved by alexcrichton

@bors
Copy link
Contributor

bors commented Jun 28, 2018

⌛ Testing commit dcff154 with merge 42a5ad8...

bors added a commit that referenced this pull request Jun 28, 2018
libc: changes to ppc64le musl branch to support building of rust on A…

…lpine

This PR includes changes to the libc musl branch to include the correct defines & declarations to support powerpc64.  Values that needed changes to a definition for powerpc64.rs that existed higher in the branch also resulted in a change that moved the definition down to the b32/mod.rs, b64/x86_64.rs to ensure that builds continued to work on those architectures.

Verification was done building rust for both ppc64le and x86_64 on Alpine as described in the git project
https://github.com/mksully22/ppc64le_alpine_rust_1.26.2
@bors
Copy link
Contributor

bors commented Jun 28, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing 42a5ad8 to master...

@bors bors merged commit dcff154 into rust-lang:master Jun 28, 2018
bors added a commit that referenced this pull request Jul 1, 2018
Re-add aarch64 stuff removed by previous PR

Previous PR #1023 has removed stuff from mod.rs and added it to some submodules, but
missed the aarch64 submodule.

This caused a build failure for rust on aarch64, see rust-lang/rust#51864.

This copies the stuff that that commit added to the x86_64.rs submodule
and puts it into aarch64.rs.

It's been tested in the rust-lang/rust PR above: https://travis-ci.org/rust-lang/rust/builds/398750336
kennytm added a commit to kennytm/rust that referenced this pull request Jul 5, 2018
rust: add initial changes to support powerpc64le musl

Initial changes to support rustc building on ppc64le with musl. A PR was also submitted to libc component rust-lang/libc#1023 to add changes to libc musl definitions.

A PR was submitted on Alpine alpinelinux/aports#4549 with equivalent temporary patches for building on Alpine for now.

As a verification test a github project was put together to build ppc64le versions of rustc, rust-stdlib, and cargo on Alpine, https://github.com/mksully22/ppc64le_alpine_rust_1.26.2
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 this pull request may close these issues.

4 participants