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

Add powerpc-unknown-linux-muslspe compile target #100860

Closed
wants to merge 1 commit into from

Conversation

BKPepe
Copy link
Contributor

@BKPepe BKPepe commented Aug 21, 2022

This is almost identical to already existing targets:

  • powerpc_unknown_linux_musl.rs
  • powerpc_unknown_linux_gnuspe.rs

Unfortunately, I could not find documentation for these targets in src/doc/rustc/src/platform-support.

It has support for PowerPC SPE (muslspe), which
can be used with GCC version up to 8. It is useful for Freescale or IBM
cores like e500.

This was verified to be working with OpenWrt build system for CZ.NIC's
Turris 1.x routers, which are using Freescale P2020, e500v2, so add it as
a Tier 3 target.

@rust-highfive
Copy link
Collaborator

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

Please see the contribution instructions for more information.

@rustbot rustbot added the T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. label Aug 21, 2022
@rust-highfive
Copy link
Collaborator

⚠️ Warning ⚠️

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 21, 2022
@rust-log-analyzer

This comment has been minimized.

Copy link
Contributor

@ehuss ehuss left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In order for the new documentation file to be picked up, it needs to be added to SUMMARY.md.

src/doc/rustc/src/platform-support.md Outdated Show resolved Hide resolved
@ehuss
Copy link
Contributor

ehuss commented Aug 21, 2022

r? compiler-team

@rust-log-analyzer

This comment has been minimized.

@BKPepe BKPepe force-pushed the powerpc-muslspe branch 2 times, most recently from 3cfd0e1 to c069457 Compare August 22, 2022 02:22
@rust-log-analyzer

This comment has been minimized.

@rust-log-analyzer

This comment has been minimized.

src/doc/rustc/src/SUMMARY.md Outdated Show resolved Hide resolved
@wesleywiser
Copy link
Member

Please quote the corresponding requirements verbatim from the tier 3 target tier policy to explain how the target meets those requirements (leaving a new comment here is fine). You can find an example of this in the PR description for #86191. Thank you!

This is almost identical to already existing targets:
- powerpc_unknown_linux_musl.rs
- powerpc_unknown_linux_gnuspe.rs

It has support for PowerPC SPE (muslspe), which
can be used with GCC version up to 8. It is useful for Freescale or IBM
cores like e500.

This was verified to be working with OpenWrt build system for CZ.NIC's
Turris 1.x routers, which are using Freescale P2020, e500v2, so add it as
a Tier 3 target.
@apiraino
Copy link
Contributor

Switching to waiting on author to incorporate changes suggested in this comment. Feel free to request a review with @rustbot ready, thanks!

@rustbot author

@rustbot rustbot added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Oct 13, 2022
@wesleywiser
Copy link
Member

Hi @BKPepe! Specifically, we're looking for you to leave a comment like this one, copying and replying to the tier 3 target policy.

Thanks!

@bors
Copy link
Contributor

bors commented Nov 23, 2022

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

@JohnCSimon
Copy link
Member

@BKPepe
ping from triage - can you post your status on this PR? There hasn't been an update in a few months. Thanks!

FYI: when a PR is ready for review, send a message containing
@rustbot ready to switch to S-waiting-on-review so the PR is in the reviewer's backlog.

@jyn514
Copy link
Member

jyn514 commented Apr 26, 2023

I'm going to close this for now since we haven't heard back in a while. Feel free to re-open the PR if you have more time in the future (make sure to re-open before pushing to the branch; due to a quirk of github you'll have to make a new PR otherwise).

@jyn514 jyn514 closed this Apr 26, 2023
@BKPepe
Copy link
Contributor Author

BKPepe commented Jul 18, 2024

It looks I will need to create a new PR, because I can not re-open this one.

matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Aug 14, 2024
Add powerpc-unknown-linux-muslspe compile target

This is almost identical to already existing targets:
- powerpc_unknown_linux_musl.rs
- powerpc_unknown_linux_gnuspe.rs

It has support for PowerPC SPE (muslspe), which
can be used with GCC version up to 8. It is useful for Freescale or IBM cores like e500.

This was verified to be working with OpenWrt build system for CZ.NIC's Turris 1.x routers, which are using Freescale P2020, e500v2, so add it as a Tier 3 target.

Follow-up of rust-lang#100860
rust-timer added a commit to rust-lang-ci/rust that referenced this pull request Aug 15, 2024
Rollup merge of rust-lang#127905 - BKPepe:powerpc-muslspe, r=wesleywiser

Add powerpc-unknown-linux-muslspe compile target

This is almost identical to already existing targets:
- powerpc_unknown_linux_musl.rs
- powerpc_unknown_linux_gnuspe.rs

It has support for PowerPC SPE (muslspe), which
can be used with GCC version up to 8. It is useful for Freescale or IBM cores like e500.

This was verified to be working with OpenWrt build system for CZ.NIC's Turris 1.x routers, which are using Freescale P2020, e500v2, so add it as a Tier 3 target.

Follow-up of rust-lang#100860
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants