Skip to content
This repository was archived by the owner on Feb 8, 2024. It is now read-only.

Add Saturation Arithmetic Intrinsics#161

Merged
kinke merged 2 commits intoldc-developers:ldcfrom
baziotis:sat_intrinsics
Aug 6, 2019
Merged

Add Saturation Arithmetic Intrinsics#161
kinke merged 2 commits intoldc-developers:ldcfrom
baziotis:sat_intrinsics

Conversation

@baziotis
Copy link
Copy Markdown

@baziotis baziotis commented Aug 6, 2019

Solving issue: ldc-developers/ldc#2925

@kinke
Copy link
Copy Markdown
Member

kinke commented Aug 6, 2019

Thx. These intrinsics have apparently been added with LLVM 8 (not present in http://releases.llvm.org/7.0.0/docs/LangRef.html, but in http://releases.llvm.org/8.0.0/docs/LangRef.html), so please guard with version (INTRINSICS_FROM_800).

@baziotis
Copy link
Copy Markdown
Author

baziotis commented Aug 6, 2019

Done, I don't know how you prefer the indentation though.

@thewilsonator
Copy link
Copy Markdown

D files in the D style, C++ in the LLVM style.

@baziotis
Copy link
Copy Markdown
Author

baziotis commented Aug 6, 2019

Yes, but I've seen version in different flavors. From what I see, I think it's ok (i.e. the other version (INTRINSICS...) follow the same indentation.

@kinke kinke merged commit adf994f into ldc-developers:ldc Aug 6, 2019
@baziotis baziotis deleted the sat_intrinsics branch August 7, 2019 13:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants