llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang-tools-extra Author: Abe (Abe149) <details> <summary>Changes</summary> mainly, fixed “the Clang module mechanism doesn’t support headers the rely on other headers” => “the Clang module mechanism doesn’t support headers that rely on other headers”. [emphasis on “the” versus “that”] --- Full diff: https://github.com/llvm/llvm-project/pull/99256.diff 1 Files Affected: - (modified) clang-tools-extra/docs/modularize.rst (+2-2) ``````````diff diff --git a/clang-tools-extra/docs/modularize.rst b/clang-tools-extra/docs/modularize.rst index 64ca8c99d4e8e..97fd33b958650 100644 --- a/clang-tools-extra/docs/modularize.rst +++ b/clang-tools-extra/docs/modularize.rst @@ -254,8 +254,8 @@ For example, with the same header list from above:: } Note that headers with dependents will be ignored with a warning, as the -Clang module mechanism doesn't support headers the rely on other headers -to be included first. +Clang module mechanism doesn't support headers that rely on other headers +being included first. The module map format defines some keywords which can't be used in module names. If a header has one of these names, an underscore ('_') will be `````````` </details> https://github.com/llvm/llvm-project/pull/99256 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
