Skip to content

Commit 2002bfe

Browse files
committed
Update a comment to match current core issues list.
llvm-svn: 193970
1 parent 8b86f2d commit 2002bfe

File tree

1 file changed

+3
-8
lines changed

1 file changed

+3
-8
lines changed

clang/lib/Sema/SemaDeclCXX.cpp

Lines changed: 3 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -5576,9 +5576,9 @@ bool Sema::SpecialMemberIsTrivial(CXXMethodDecl *MD, CXXSpecialMember CSM,
55765576

55775577
bool ConstArg = false;
55785578

5579-
// C++11 [class.copy]p12, p25:
5580-
// A [special member] is trivial if its declared parameter type is the same
5581-
// as if it had been implicitly declared [...]
5579+
// C++11 [class.copy]p12, p25: [DR1593]
5580+
// A [special member] is trivial if [...] its parameter-type-list is
5581+
// equivalent to the parameter-type-list of an implicit declaration [...]
55825582
switch (CSM) {
55835583
case CXXDefaultConstructor:
55845584
case CXXDestructor:
@@ -5622,11 +5622,6 @@ bool Sema::SpecialMemberIsTrivial(CXXMethodDecl *MD, CXXSpecialMember CSM,
56225622
llvm_unreachable("not a special member");
56235623
}
56245624

5625-
// FIXME: We require that the parameter-declaration-clause is equivalent to
5626-
// that of an implicit declaration, not just that the declared parameter type
5627-
// matches, in order to prevent absuridities like a function simultaneously
5628-
// being a trivial copy constructor and a non-trivial default constructor.
5629-
// This issue has not yet been assigned a core issue number.
56305625
if (MD->getMinRequiredArguments() < MD->getNumParams()) {
56315626
if (Diagnose)
56325627
Diag(MD->getParamDecl(MD->getMinRequiredArguments())->getLocation(),

0 commit comments

Comments
 (0)