[PATCH] D131541: [Sema] Fix friend destructor declarations after D130936
Roy Jacobson via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sat Aug 13 12:35:32 PDT 2022
royjacobson updated this revision to Diff 452447.
royjacobson added a comment.
Update to handle some edge cases better.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D131541/new/
https://reviews.llvm.org/D131541
Files:
clang/lib/Sema/SemaDecl.cpp
clang/test/SemaCXX/member-class-11.cpp
Index: clang/test/SemaCXX/member-class-11.cpp
===================================================================
--- clang/test/SemaCXX/member-class-11.cpp
+++ clang/test/SemaCXX/member-class-11.cpp
@@ -26,4 +26,51 @@
~B(); // expected-error {{expected the class name after '~' to name the enclosing class}}
};
+template <typename T>
+struct D {
+ friend T::S::~S();
+private:
+ static constexpr int secret = 42;
+};
+
+template <typename T>
+struct E {
+ friend T::S::~V();
+};
+
+struct BadInstantiation {
+ struct S {
+ struct V {};
+ };
+};
+
+struct GoodInstantiation {
+ struct V {
+ ~V();
+ };
+ using S = V;
+};
+
+// FIXME: We should diagnose this while instantiating.
+E<BadInstantiation> x;
+E<GoodInstantiation> y;
+
+struct Q {
+ struct S { ~S(); };
+};
+
+Q::S::~S() {
+ void foo(int);
+ foo(D<Q>::secret);
+}
+
+struct X {
+ ~X();
+};
+struct Y;
+
+struct Z {
+ friend X::~Y(); // expected-error {{expected the class name after '~' to name the enclosing class}}
+};
+
}
Index: clang/lib/Sema/SemaDecl.cpp
===================================================================
--- clang/lib/Sema/SemaDecl.cpp
+++ clang/lib/Sema/SemaDecl.cpp
@@ -11514,16 +11514,25 @@
if (CXXConstructorDecl *Constructor = dyn_cast<CXXConstructorDecl>(NewFD)) {
CheckConstructor(Constructor);
} else if (CXXDestructorDecl *Destructor =
- dyn_cast<CXXDestructorDecl>(NewFD)) {
- CXXRecordDecl *Record = Destructor->getParent();
- QualType ClassType = Context.getTypeDeclType(Record);
-
- DeclarationName Name = Context.DeclarationNames.getCXXDestructorName(
- Context.getCanonicalType(ClassType));
- if (NewFD->getDeclName() != Name) {
- Diag(NewFD->getLocation(), diag::err_destructor_name);
- NewFD->setInvalidDecl();
- return Redeclaration;
+ dyn_cast<CXXDestructorDecl>(NewFD)) {
+ // We check here for invalid destructor names.
+ // If we have a friend destructor declaration that is dependent, we can't
+ // diagnose right away because cases like this are still valid:
+ // template <class T> struct A { friend T::X::~Y(); };
+ // struct B { struct Y { ~Y(); }; using X = Y; };
+ // template struct A<B>;
+ if (NewFD->getFriendObjectKind() == Decl::FriendObjectKind::FOK_None ||
+ !Destructor->getThisType()->isDependentType()) {
+ CXXRecordDecl *Record = Destructor->getParent();
+ QualType ClassType = Context.getTypeDeclType(Record);
+
+ DeclarationName Name = Context.DeclarationNames.getCXXDestructorName(
+ Context.getCanonicalType(ClassType));
+ if (NewFD->getDeclName() != Name) {
+ Diag(NewFD->getLocation(), diag::err_destructor_name);
+ NewFD->setInvalidDecl();
+ return Redeclaration;
+ }
}
} else if (auto *Guide = dyn_cast<CXXDeductionGuideDecl>(NewFD)) {
if (auto *TD = Guide->getDescribedFunctionTemplate())
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D131541.452447.patch
Type: text/x-patch
Size: 2985 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20220813/0463dd6e/attachment.bin>
More information about the cfe-commits
mailing list