-
Notifications
You must be signed in to change notification settings - Fork 5k
Include xmmintrin.h early on x86 as well #113279
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.
Tagging subscribers to this area: @mangod9 |
The other error is:
|
Thanks for your help @am11! |
@jkoritzinsky I think we can remove extern "C" from this line runtime/src/tests/Interop/DllImportAttribute/DllImportPath/DllImportPathNative.cpp Line 13 in 2ec50bd
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @jkoritzinsky and @am11!
We're hitting problems with our
debugreturn.h
machinery on x86 with Clang 20. Include xmmintrin.h early on that platform as well to work around it.Related: #113248