Skip to content

Conversation

@kazutakahirata
Copy link
Contributor

Lambda is already of CXXRecordDecl *.

Lambda is already of CXXRecordDecl *.
@llvmbot llvmbot added clang Clang issues not falling into any other category clang:modules C++20 modules and Clang Header Modules labels Jul 4, 2025
@llvmbot
Copy link
Member

llvmbot commented Jul 4, 2025

@llvm/pr-subscribers-clang

@llvm/pr-subscribers-clang-modules

Author: Kazu Hirata (kazutakahirata)

Changes

Lambda is already of CXXRecordDecl *.


Full diff: https://github.com/llvm/llvm-project/pull/146986.diff

1 Files Affected:

  • (modified) clang/lib/Serialization/ASTReader.cpp (+1-2)
diff --git a/clang/lib/Serialization/ASTReader.cpp b/clang/lib/Serialization/ASTReader.cpp
index d380afd1ed1bd..6e03de87e587d 100644
--- a/clang/lib/Serialization/ASTReader.cpp
+++ b/clang/lib/Serialization/ASTReader.cpp
@@ -9463,8 +9463,7 @@ void ASTReader::AssignedLambdaNumbering(CXXRecordDecl *Lambda) {
 
   // Keep track of this lambda so it can be merged with another lambda that
   // is loaded later.
-  LambdaDeclarationsForMerging.insert(
-      {LambdaInfo, const_cast<CXXRecordDecl *>(Lambda)});
+  LambdaDeclarationsForMerging.insert({LambdaInfo, Lambda});
 }
 
 void ASTReader::LoadSelector(Selector Sel) {

@kazutakahirata kazutakahirata changed the title [Lambda] Remove an unnecessary cast (NFC) [Serialization] Remove an unnecessary cast (NFC) Jul 4, 2025
@kazutakahirata kazutakahirata merged commit 8fe205d into llvm:main Jul 4, 2025
12 checks passed
@kazutakahirata kazutakahirata deleted the cleanup_20250703_cast_clang_Serialization branch July 4, 2025 05:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

clang:modules C++20 modules and Clang Header Modules clang Clang issues not falling into any other category

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants