llvmorg-github-actions[bot] wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang Author: Julian Nagele (juliannagele) <details> <summary>Changes</summary> https://github.com/llvm/llvm-project/pull/203214 added BuiltinsAVR.def but omitted its textual header entry in module.modulemap. Building the Clang_Basic module then tried to import the .def as a submodule from inside namespace clang::AVR, which is illegal, breaking the module build. Add the missing entry. --- Full diff: https://github.com/llvm/llvm-project/pull/204584.diff 1 Files Affected: - (modified) clang/include/module.modulemap (+1) ``````````diff diff --git a/clang/include/module.modulemap b/clang/include/module.modulemap index b5376e840bce2..d3f616b3fd636 100644 --- a/clang/include/module.modulemap +++ b/clang/include/module.modulemap @@ -42,6 +42,7 @@ module Clang_Basic { textual header "clang/Basic/BuiltinsAArch64NeonSVEBridge.def" textual header "clang/Basic/BuiltinsAArch64NeonSVEBridge_cg.def" textual header "clang/Basic/BuiltinsARM.def" + textual header "clang/Basic/BuiltinsAVR.def" textual header "clang/Basic/BuiltinsHexagonMapCustomDep.def" textual header "clang/Basic/BuiltinsLoongArchBase.def" textual header "clang/Basic/BuiltinsLoongArchLASX.def" `````````` </details> https://github.com/llvm/llvm-project/pull/204584 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
