-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Hardware Intrinsics: Add uint overload for Sse41.MultiplyLow #10675
Comments
CC. @fiigii, @CarolEidt, @eerhardt |
@Zhentar Thanks for the proposal and I agree. @tannergooding I am also considering to add all the integer overloads for |
@fiigii, We'll want to make sure we have an issue tracking it and that it goes to API review (I am working on ensuring another one is setup soon to cover the issues/requrests that have cropped up). |
I see. Let me open an issue for |
Please close this issue. |
Works, but it's pretty ugly.
(for the benefit of those, who like me from two days ago, have minds happily free of two's complement arithmetic, the low half of a multiply is the same for both signed and unsigned values. So while Intel documents the underlying PMULLD as signed, it is actually sign agnostic)
I suspect that there is a matching
Avx2.MultiplyLow
for which this is equally applicable.The text was updated successfully, but these errors were encountered: