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

__umulh, __mulh, _addcarry_u32, and _addcarry_u64 #1240

Open
perdrix52 opened this issue Dec 2, 2024 · 1 comment
Open

__umulh, __mulh, _addcarry_u32, and _addcarry_u64 #1240

perdrix52 opened this issue Dec 2, 2024 · 1 comment

Comments

@perdrix52
Copy link

perdrix52 commented Dec 2, 2024

Background: The Qt framework (in qnumeric.h) includes <intrin.h> when compiling with MSVC. This header is included if any code uses any Qt classes. Unfortunately this breaks simde with lots of errors about redefinitions.

Does simde provide a header that defines these intrinsics in a portable manner such that I could modify their header to read e.g.

#if __has_include("simde-xxxxx.h")
#include "simde-xxxxx.h"
#else
#include <intrin.h>
#endif

@mr-c
Copy link
Collaborator

mr-c commented Dec 2, 2024

Hello @perdrix52 , SIMDe does not yet have implementations for __umulh, __mulh, _addcarry_u32, and _addcarry_u64. I would be happy to review such a contribution.

Can you share more about the errors you are seeing?

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

No branches or pull requests

2 participants