Skip to content

Header <intrin.h> is incompatible with Clang 19 #443

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.

Sign up for GitHub

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

Jump to bottom
Closed
zufuliu opened this issue Jul 27, 2024 · 3 comments
Closed

Header <intrin.h> is incompatible with Clang 19 #443

zufuliu opened this issue Jul 27, 2024 · 3 comments

Comments

@zufuliu
Copy link

zufuliu commented Jul 27, 2024

test case:

#include <intrin.h>
D:\Dev\llvm-mingw-20240726-ucrt-x86_64\bin>clang.exe -c test.c
In file included from test.c:1:
In file included from D:/Dev/llvm-mingw-20240726-ucrt-x86_64/lib/clang/19/include/intrin.h:12:
In file included from D:/Dev/llvm-mingw-20240726-ucrt-x86_64/include/intrin.h:122:
D:/Dev/llvm-mingw-20240726-ucrt-x86_64/lib/clang/19/include/mm3dnow.h:16:2: warning: "The <mm3dnow.h> header is
      deprecated, and 3dNow! intrinsics are unsupported. For other intrinsics, include <x86intrin.h>, instead."
      [-W#warnings]
   16 | #warning "The <mm3dnow.h> header is deprecated, and 3dNow! intrinsics are unsupported. For other intrinsics, inc...
      |  ^
1 warning generated.

D:\Dev\llvm-mingw-20240726-ucrt-x86_64\bin>clang.exe --target=x86_64-pc-windows-msvc -c test.c
@mstorsjo
Copy link
Owner

Thanks for the report, I sent a patch to fix this issue at https://sourceforge.net/p/mingw-w64/mailman/message/58800445/.

@zufuliu
Copy link
Author

zufuliu commented Jul 29, 2024

Thanks for the fix 👍

github-actions bot pushed a commit to smalltalkman/mingw-w64 that referenced this issue Jul 29, 2024
@mstorsjo
headers: Stop including mm3dnow.h in intrin.h
36c6211 
Clang 19 has removed all support for 3dnow, see
llvm/llvm-project@f0eb558
and
llvm/llvm-project#96246.

Since this commit, including the mm3dnow.h header still works,
but it produces a warning about the header being deprecated.

As 3DNow! has been deprecated for a very long time, stop
including this header, for all compiler configurations.

This issue was reported in
mstorsjo/llvm-mingw#443.

Signed-off-by: Martin Storsjö <[email protected]>
@zufuliu
Copy link
Author

zufuliu commented Jul 30, 2024

Closed this as the patch has been committed.

@zufuliu zufuliu closed this as completed Jul 30, 2024
nak5124 pushed a commit to nak5124/MinGW-ld64 that referenced this issue Jul 31, 2024
@mstorsjo @nak5124
headers: Stop including mm3dnow.h in intrin.h
d195a85 
Clang 19 has removed all support for 3dnow, see
llvm/llvm-project@f0eb558
and
llvm/llvm-project#96246.

Since this commit, including the mm3dnow.h header still works,
but it produces a warning about the header being deprecated.

As 3DNow! has been deprecated for a very long time, stop
including this header, for all compiler configurations.

This issue was reported in
mstorsjo/llvm-mingw#443.

Signed-off-by: Martin Storsjö <[email protected]>
nak5124 pushed a commit to nak5124/MinGW-ld64 that referenced this issue Jul 31, 2024
@mstorsjo @nak5124
headers: Stop including mm3dnow.h in intrin.h
da78b25 
Clang 19 has removed all support for 3dnow, see
llvm/llvm-project@f0eb558
and
llvm/llvm-project#96246.

Since this commit, including the mm3dnow.h header still works,
but it produces a warning about the header being deprecated.

As 3DNow! has been deprecated for a very long time, stop
including this header, for all compiler configurations.

This issue was reported in
mstorsjo/llvm-mingw#443.

Signed-off-by: Martin Storsjö <[email protected]>
jaavid pushed a commit to J-Androids/superproject that referenced this issue Feb 10, 2025
@rprichard
headers: Stop including mm3dnow.h in intrin.h
df7adeb 
Clang 19 has removed all support for 3dnow, see
llvm/llvm-project@f0eb558
and
llvm/llvm-project#96246.

Since this commit, including the mm3dnow.h header still works,
but it produces a warning about the header being deprecated.

As 3DNow! has been deprecated for a very long time, stop
including this header, for all compiler configurations.

This issue was reported in
mstorsjo/llvm-mingw#443.

Cherry-picked from
mirror/mingw-w64@36c6211

Bug: http://b/388603030
Test: presubmit
Signed-off-by: Martin Storsjö <[email protected]>
Change-Id: Ieac324c6ef8b62c2eb2b010376ff1cc845901f4b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mstorsjo @zufuliu