llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-backend-x86 Author: Cyndy Ishida (cyndyishida) <details> <summary>Changes</summary> Since resource headers are installed, commonly as system dependencies, they should have `[system]` on them. Fix up the ones missing. --- Full diff: https://github.com/llvm/llvm-project/pull/156934.diff 1 Files Affected: - (modified) clang/lib/Headers/module.modulemap (+2-2) ``````````diff diff --git a/clang/lib/Headers/module.modulemap b/clang/lib/Headers/module.modulemap index a72828625a629..bdf5119ba4607 100644 --- a/clang/lib/Headers/module.modulemap +++ b/clang/lib/Headers/module.modulemap @@ -329,13 +329,13 @@ module _Builtin_unwind [system] { } // End -fbuiltin-headers-in-system-modules affected modules -module opencl_c { +module opencl_c [system] { requires opencl header "opencl-c.h" header "opencl-c-base.h" } -module ptrauth { +module ptrauth [system] { header "ptrauth.h" export * } `````````` </details> https://github.com/llvm/llvm-project/pull/156934 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
