================
@@ -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;
----------------
a-tarasyuk wrote:
The weird thing is that this is a valid decoration -
https://compiler-explorer.com/z/fKoozKb3f
For instance currently to check lambda implicit object parameter, used opposite
to explicit object member function with NULL validation
https://github.com/llvm/llvm-project/blob/f87484d5910c1c708bfd93ef588d6ff8307e2477/clang/include/clang/AST/ASTLambda.h#L43-L51
https://github.com/llvm/llvm-project/pull/114813
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
