Kill FunctionDecl's IsCopyAssignment bit; it duplicated what could
already be determined by isCopyAssignmentOperator(), and was set too
late in the process for all clients to see the appropriate
value. Cleanup only; no functionality change.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@114916 91177308-0d34-0410-b5e6-96231b3b80d8
diff --git a/lib/AST/DeclCXX.cpp b/lib/AST/DeclCXX.cpp
index 1b89ab6..bf1e58c 100644
--- a/lib/AST/DeclCXX.cpp
+++ b/lib/AST/DeclCXX.cpp
@@ -284,10 +284,8 @@
// If this is the implicit copy constructor, note that we have now
// declared it.
// FIXME: Move constructors
- if (Method->getOverloadedOperator() == OO_Equal) {
+ if (Method->getOverloadedOperator() == OO_Equal)
data().DeclaredCopyAssignment = true;
- Method->setCopyAssignment(true);
- }
}
// Nothing else to do for implicitly-declared members.
@@ -359,8 +357,7 @@
return;
// This is a copy assignment operator.
- // Note on the decl that it is a copy assignment operator.
- Method->setCopyAssignment(true);
+ // FIXME: Move assignment operators.
// Suppress the implicit declaration of a copy constructor.
data().UserDeclaredCopyAssignment = true;