[clang] [clang][modules] Allow module maps with textual headers to be non-affecting (PR #89441)

via cfe-commits cfe-commits at lists.llvm.org
Tue Apr 23 13:39:57 PDT 2024


github-actions[bot] wrote:

<!--LLVM CODE FORMAT COMMENT: {clang-format}-->


:warning: C/C++ code formatter, clang-format found issues in your code. :warning:

<details>
<summary>
You can test this locally with the following command:
</summary>

``````````bash
git-clang-format --diff 3ea5dff0efdbb4bf5590d882810aa42f9ec26e4e ac069f486a197221adb6520d76ac8aaa74995947 -- clang/test/Modules/prune-non-affecting-module-map-files-textual.c clang/lib/Serialization/ASTWriter.cpp
``````````

</details>

<details>
<summary>
View the diff from clang-format here.
</summary>

``````````diff
diff --git a/clang/lib/Serialization/ASTWriter.cpp b/clang/lib/Serialization/ASTWriter.cpp
index b71b0150c4..4265ab42ed 100644
--- a/clang/lib/Serialization/ASTWriter.cpp
+++ b/clang/lib/Serialization/ASTWriter.cpp
@@ -213,7 +213,7 @@ GetAffectingModuleMaps(const Preprocessor &PP, Module *RootModule) {
   std::queue<const Module *> Q;
   Q.push(RootModule);
   while (!Q.empty()) {
-    const Module * CurrentModule = Q.front();
+    const Module *CurrentModule = Q.front();
     Q.pop();
 
     CollectIncludingMapsFromAncestors(CurrentModule);

``````````

</details>


https://github.com/llvm/llvm-project/pull/89441


More information about the cfe-commits mailing list