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.

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

Remove generic Intel hardware intrinsic and explode them on all the supported types #11122

Closed
fiigii opened this issue Sep 19, 2018 · 3 comments · Fixed by dotnet/coreclr#20310

Comments

@fiigii
Copy link
Contributor

fiigii commented Sep 19, 2018

According to the latest API review meeting, we determined that hardware intrinsic should explicitly be exploded on all the supported types (e.g., Vector256<int>, Vector128<long>, Vector128<short>, Vector256<float>, etc). This also makes it clear when/if support for new types (e.g., Half) are added in the future that they aren't supported in the older ISAs. But we may be able to have generic helper intrinsic (like SetAllVector256<T>).

@CarolEidt @tannergooding @eerhardt

@tannergooding
Copy link
Member

CC. @terrajobst

@Wraith2
Copy link
Contributor

Wraith2 commented Sep 20, 2018

Are api reviews still recorded or noted somewhere?

@fiigii
Copy link
Contributor Author

fiigii commented Sep 20, 2018

@Wraith2 https://github.com/dotnet/apireviews/tree/master/2018/Hardware-Intrinsics-Intel

@msftgits msftgits transferred this issue from dotnet/coreclr Jan 31, 2020
@ghost ghost locked as resolved and limited conversation to collaborators Dec 15, 2020
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 a pull request may close this issue.

3 participants