[clang] Adjust modulemap to mark mm3dnow as textual header. (PR #107155)
via cfe-commits
cfe-commits at lists.llvm.org
Tue Sep 3 14:37:56 PDT 2024
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-backend-x86
Author: James Y Knight (jyknight)
<details>
<summary>Changes</summary>
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 https://github.com/llvm/llvm-project/pull/96246#issuecomment-2322453809
---
Full diff: https://github.com/llvm/llvm-project/pull/107155.diff
1 Files Affected:
- (modified) clang/lib/Headers/module.modulemap (+2-4)
``````````diff
diff --git a/clang/lib/Headers/module.modulemap b/clang/lib/Headers/module.modulemap
index 9ffc249c8d1a23..dcaf09e8f2c558 100644
--- a/clang/lib/Headers/module.modulemap
+++ b/clang/lib/Headers/module.modulemap
@@ -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"
@@ -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
``````````
</details>
https://github.com/llvm/llvm-project/pull/107155
More information about the cfe-commits
mailing list