Skip to content

Add more standard c lib llvm intrinsics. #4067

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

Merged
merged 1 commit into from
Dec 5, 2012
Merged

Conversation

luqmana
Copy link
Member

@luqmana luqmana commented Nov 29, 2012

Expose some of the available llvm intrinsics to rust. Perhaps could use them in place of calls to libm.

@brson
Copy link
Contributor

brson commented Nov 30, 2012

Thanks!

Sorry to inconvenience you but Rust is in the process of switching to a dual MIT/ASL2 license, a fact which is not yet reflected in the Rust LICENSE file. As such I need to confirm that you consent to this license.

ASL2 is a permissive license with a patent non-aggression clause. The details are here.

If you agree just say 'I agree to license this code under the terms of MIT/ASL2'.

@brson
Copy link
Contributor

brson commented Nov 30, 2012

I guess there's no harm in doing this and somebody will need some of them eventually, though we have typically only added intrinsics as needed. Do you have a use case you need some of these for?

@luqmana
Copy link
Member Author

luqmana commented Nov 30, 2012

Ah, I've already responded in Graydon's email (i'm fine with the relicensing).

One specific use case I was thinking of was the lmath library. They also could possibly be used in the cmath lib.

@brson brson merged commit c6b1739 into rust-lang:incoming Dec 5, 2012
@brson
Copy link
Contributor

brson commented Dec 5, 2012

Merged. Thanks.

RalfJung added a commit to RalfJung/rust that referenced this pull request Apr 10, 2025
Support getting file metadata on Windows
RalfJung added a commit to RalfJung/rust that referenced this pull request Apr 10, 2025
Support getting file metadata on Windows
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.

2 participants