[clang] [C++20][Modules] Do not update the declaration generation number if the redeclaration chain completion was delayed. (PR #129982)

via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 14 14:37:46 PDT 2025


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 fbf0276b6a7a7a4508c373cf87fc349569652659 fb2de5d3725588acace54cfa62785e480c2666e0 --extensions h,cpp -- clang/test/Modules/pr121245.cpp clang/test/Modules/pr129982.cpp clang/include/clang/Serialization/ASTReader.h clang/lib/Serialization/ASTReader.cpp clang/lib/Serialization/ASTReaderDecl.cpp
``````````

</details>

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

``````````diff
diff --git a/clang/lib/Serialization/ASTReader.cpp b/clang/lib/Serialization/ASTReader.cpp
index e3a51151c2..9d7be97b2c 100644
--- a/clang/lib/Serialization/ASTReader.cpp
+++ b/clang/lib/Serialization/ASTReader.cpp
@@ -10820,7 +10820,8 @@ void ASTReader::FinishedDeserializing() {
         PendingDeducedTypeUpdates.clear();
         for (auto Update : DTUpdates) {
           ProcessingUpdatesRAIIObj ProcessingUpdates(*this);
-          // FIXME: If the return type is already deduced, check that it matches.
+          // FIXME: If the return type is already deduced, check that it
+          // matches.
           getContext().adjustDeducedFunctionResultType(Update.first,
                                                        Update.second);
         }

``````````

</details>


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


More information about the cfe-commits mailing list