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.

Sign up for GitHub

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

Jump to bottom

Add LLVM saturation intrinsics #2925

Closed
JohanEngelen opened this issue Nov 26, 2018 · 4 comments
Closed

Add LLVM saturation intrinsics #2925

JohanEngelen opened this issue Nov 26, 2018 · 4 comments
Labels
good first issue llvm Related to LLVM

Comments

@JohanEngelen
Copy link
Member

See: https://reviews.llvm.org/rL347334

See druntime's ldc/intrinsics.di.
@kinke
Copy link
Member

kinke commented Nov 26, 2018

While at it, there's also new llvm.{maximum,minimum} with slightly different semantics than existing llvm.{min,max}num.

@JohanEngelen JohanEngelen added the llvm Related to LLVM label Dec 12, 2018
@baziotis baziotis mentioned this issue Aug 6, 2019
Merged
@baziotis
Copy link
Contributor

baziotis commented Aug 7, 2019

I think better close this issue.

@kinke
Copy link
Member

kinke commented Aug 7, 2019

I left it open because of maximum, minimum and may finally get to it soon, adding it should be a matter of minutes.

@baziotis
Copy link
Contributor

baziotis commented Aug 7, 2019 •
edited
Loading

Right, I missed those.
This: https://llvm.org/docs/LangRef.html#llvm-minimum-intrinsic
and this: https://llvm.org/docs/LangRef.html#llvm-maximum-intrinsic
right?

Edit: PR: ldc-developers/druntime#162

@kinke kinke closed this as completed Aug 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue llvm Related to LLVM
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@kinke @JohanEngelen @baziotis