Skip to content

Conversation

cuviper
Copy link
Member

@cuviper cuviper commented Jul 28, 2017

Hard-float ARM targets use the AAPCS-VFP ABI, which passes and returns
homogeneous float/vector aggregates in the VFP registers.

Fixes #43329.

r? @eddyb

Hard-float ARM targets use the AACPS-VFP ABI, which passes and returns
homogeneous float/vector aggregates in the VFP registers.

Fixes rust-lang#43329.
@eddyb
Copy link
Member

eddyb commented Jul 28, 2017

@bors r+ Thanks!

@bors
Copy link
Collaborator

bors commented Jul 28, 2017

📌 Commit efc6764 has been approved by eddyb

@shepmaster shepmaster added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Jul 28, 2017
@bors
Copy link
Collaborator

bors commented Jul 29, 2017

⌛ Testing commit efc6764 with merge 42a09c0...

bors added a commit that referenced this pull request Jul 29, 2017
Support homogeneous aggregates for hard-float ARM

Hard-float ARM targets use the AAPCS-VFP ABI, which passes and returns
homogeneous float/vector aggregates in the VFP registers.

Fixes #43329.

r? @eddyb
@bors
Copy link
Collaborator

bors commented Jul 29, 2017

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

@bors bors merged commit efc6764 into rust-lang:master Jul 29, 2017
@alexcrichton
Copy link
Member

@cuviper do you think we should backport this?

@cuviper
Copy link
Member Author

cuviper commented Jul 29, 2017

I don't think it's critical, but it's an easy cherry-pick if we want it. Note that it wasn't actually a regression, just a new case added to the existing test, which never would have worked on arm hf before.

@alexcrichton
Copy link
Member

Ok, sounds like it's ok to ride the trains.

@cuviper cuviper deleted the aapcs_vfp branch July 29, 2017 05:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants