Skip to content

Conversation

@teskje
Copy link
Contributor

@teskje teskje commented Nov 26, 2017

mulsf3 and muldf3 have Rust implementations since 8970485. This PR removes the C implementations to avoid conflicts.

Without this I wasn't able to compile the crate with the "c" feature enabled, the errors being
multiple definition of '__muldf3' and multiple definition of '__mulsf3'.

mulsf3 and muldf3 have Rust implementations since 8970485. Remove the
C implementations to avoid conflicts.
@alexcrichton
Copy link
Member

@bors: r+

@bors
Copy link
Contributor

bors commented Nov 26, 2017

📌 Commit 0474060 has been approved by alexcrichton

@bors
Copy link
Contributor

bors commented Nov 26, 2017

⌛ Testing commit 0474060 with merge 18feacc...

bors added a commit that referenced this pull request Nov 26, 2017
…xcrichton

Remove mulsf3.c and muldf3.c from compiler-rt.

mulsf3 and muldf3 have Rust implementations since 8970485. This PR removes the C implementations to avoid conflicts.

Without this I wasn't able to compile the crate with the "c" feature enabled, the errors being
`multiple definition of '__muldf3'` and `multiple definition of '__mulsf3'`.
@bors
Copy link
Contributor

bors commented Nov 26, 2017

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

@bors bors merged commit 0474060 into rust-lang:master Nov 26, 2017
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.

3 participants