Skip to content

Commit

Permalink
Adjust modulemap to mark mm3dnow as textual header. (#107155)
Browse files Browse the repository at this point in the history
This avoids issuing the deprecation diagnostic when building the module.

Not building it into a module shouldn't cause any negative impacts,
since it no longer has any declarations other than the header guard.
It's also very rarely included by anything.

Addresses
#96246 (comment)
  • Loading branch information
jyknight authored Sep 13, 2024
1 parent 8e2843b commit f902339
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions clang/lib/Headers/module.modulemap
Original file line number Diff line number Diff line change
Expand Up @@ -66,6 +66,8 @@ module _Builtin_intrinsics [system] [extern_c] {
textual header "__wmmintrin_aes.h"
textual header "__wmmintrin_pclmul.h"

textual header "mm3dnow.h"

explicit module mm_malloc {
requires !freestanding
header "mm_malloc.h"
Expand Down Expand Up @@ -122,10 +124,6 @@ module _Builtin_intrinsics [system] [extern_c] {
header "popcntintrin.h"
}

explicit module mm3dnow {
header "mm3dnow.h"
}

explicit module aes_pclmul {
header "wmmintrin.h"
export aes
Expand Down

0 comments on commit f902339

Please sign in to comment.