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 armv4t-unknown-linux-gnueabi target #47018

Merged
merged 2 commits into from
Dec 28, 2017
Merged

Conversation

malbarbo
Copy link
Contributor

@malbarbo malbarbo commented Dec 26, 2017

armv4t was left out of #37615 (armv5te addition) to be included in a future PR. So this PR adds armv4t target.

armv4t target is useful because the armel port of Debian targets armv4t

@rust-highfive
Copy link
Collaborator

r? @arielb1

(rust_highfive has picked a reviewer for you, use r? to override)

@malbarbo
Copy link
Contributor Author

This was copied from armv5te and the cpu changed to armv4t;

@malbarbo malbarbo changed the title [WIP] Add armv4t-unknown-linux-gnueabi target Add armv4t-unknown-linux-gnueabi target Dec 27, 2017
@malbarbo
Copy link
Contributor Author

r? @alexcrichton

@kennytm kennytm added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Dec 27, 2017
@alexcrichton
Copy link
Member

@bors: r+

@bors
Copy link
Contributor

bors commented Dec 27, 2017

📌 Commit 36d486a has been approved by alexcrichton

@bors
Copy link
Contributor

bors commented Dec 28, 2017

⌛ Testing commit 36d486a with merge 9a8c753...

bors added a commit that referenced this pull request Dec 28, 2017
Add armv4t-unknown-linux-gnueabi target

armv4t was left out of #37615 (armv5te addition) to be included in a [future PR](#37615 (comment)).  So this PR adds armv4t target.

armv4t target is useful because the [armel](https://wiki.debian.org/ArmEabiPort) port of Debian targets armv4t
@bors
Copy link
Contributor

bors commented Dec 28, 2017

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

@bors bors merged commit 36d486a into rust-lang:master Dec 28, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants