Author: erichkeane Date: 2025-03-12T08:19:21-07:00 New Revision: 5c02e74d21fdec44fa9c3a8b45ef60e5b433bf71
URL: https://github.com/llvm/llvm-project/commit/5c02e74d21fdec44fa9c3a8b45ef60e5b433bf71 DIFF: https://github.com/llvm/llvm-project/commit/5c02e74d21fdec44fa9c3a8b45ef60e5b433bf71.diff LOG: Fix use of CXXThisScopeRAII Seemingly I managed to not give this a name, and not notice that it didn't properly introduce the scope it was supposed to! This patch gives it a name which should hopefully/presumably fix any cases where we don't properly introduce the 'this' scope. I presume that previous callers to this might also do this in most cases so that this is a redundant scope, but we have to make sure it happens. FIXES: #130846 Added: Modified: clang/lib/Sema/SemaTemplate.cpp Removed: ################################################################################ diff --git a/clang/lib/Sema/SemaTemplate.cpp b/clang/lib/Sema/SemaTemplate.cpp index 64aabb1fcdc35..a56720ec26fbd 100644 --- a/clang/lib/Sema/SemaTemplate.cpp +++ b/clang/lib/Sema/SemaTemplate.cpp @@ -5845,7 +5845,7 @@ bool Sema::CheckTemplateArgumentList( ThisQuals = Method->getMethodQualifiers(); ContextRAII Context(*this, NewContext); - CXXThisScopeRAII(*this, RD, ThisQuals, RD != nullptr); + CXXThisScopeRAII Scope(*this, RD, ThisQuals, RD != nullptr); MultiLevelTemplateArgumentList MLTAL = getTemplateInstantiationArgs( Template, NewContext, /*Final=*/false, CTAI.CanonicalConverted, _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
