Skip to content

Commit 6e9c833

Browse files
erichkeanefrederik-h
authored andcommitted
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: llvm#130846
1 parent c14dc34 commit 6e9c833

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

clang/lib/Sema/SemaTemplate.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5845,7 +5845,7 @@ bool Sema::CheckTemplateArgumentList(
58455845
ThisQuals = Method->getMethodQualifiers();
58465846

58475847
ContextRAII Context(*this, NewContext);
5848-
CXXThisScopeRAII(*this, RD, ThisQuals, RD != nullptr);
5848+
CXXThisScopeRAII Scope(*this, RD, ThisQuals, RD != nullptr);
58495849

58505850
MultiLevelTemplateArgumentList MLTAL = getTemplateInstantiationArgs(
58515851
Template, NewContext, /*Final=*/false, CTAI.CanonicalConverted,

0 commit comments

Comments
 (0)