[clang] [Clang] eliminate shadowing warnings for parameters using deducing this (PR #114813)
via cfe-commits
cfe-commits at lists.llvm.org
Tue Nov 5 23:13:13 PST 2024
================
@@ -8236,11 +8236,14 @@ void Sema::CheckShadow(NamedDecl *D, NamedDecl *ShadowedDecl,
DeclContext *NewDC = D->getDeclContext();
if (FieldDecl *FD = dyn_cast<FieldDecl>(ShadowedDecl)) {
- // Fields are not shadowed by variables in C++ static methods.
- if (CXXMethodDecl *MD = dyn_cast<CXXMethodDecl>(NewDC))
+ if (CXXMethodDecl *MD = dyn_cast<CXXMethodDecl>(NewDC)) {
+ // Fields are not shadowed by variables in C++ static methods.
if (MD->isStatic())
return;
+ if (!MD->getParent()->isLambda() && MD->isExplicitObjectMemberFunction())
+ return;
----------------
Sirraide wrote:
> Thanks for bringing that to my attention. In such cases
>
> ```c++
> struct C {
> int b = 5;
> int foo() {
> return [a = b]() {
> return [=, b = a]() {
> return b;
> }();
> }();
> }
> };
> ```
>
> depending exclusively on `isExplicitObjectMemberFunction` isn’t safe, as it could lead to assertion failures during type resolution
Er, can you elaborate what the problem here is maybe?
https://github.com/llvm/llvm-project/pull/114813
More information about the cfe-commits
mailing list