Eliminate Superfluous Trivial Copy Constructors

These force an implicit copy-assignment operator to be generated which is a deprecated behavior in modern C++.

Author: CodaFi

include/swift/AST/PrintOptions.h

@@ -95,7 +95,6 @@ class AnyAttrKind {
     static_assert(DAK_Count < UINT_MAX, "DeclAttrKind is > 31 bits");
   }
   AnyAttrKind() : kind(TAK_Count), isType(1) {}
-  AnyAttrKind(const AnyAttrKind &) = default;
 
   /// Returns the TypeAttrKind, or TAK_Count if this is not a type attribute.
   TypeAttrKind type() const {

include/swift/Basic/SupplementaryOutputPaths.h

@@ -156,7 +156,6 @@ struct SupplementaryOutputPaths {
   std::string BitstreamOptRecordPath;
 
   SupplementaryOutputPaths() = default;
-  SupplementaryOutputPaths(const SupplementaryOutputPaths &) = default;
 
   /// Apply a given function for each existing (non-empty string) supplementary output
   void forEachSetOutput(llvm::function_ref<void(const std::string&)> fn) const {

include/swift/Parse/ParserPosition.h

@@ -31,7 +31,6 @@ class ParserPosition {
       : LS(LS), PreviousLoc(PreviousLoc) {}
 public:
   ParserPosition() = default;
-  ParserPosition &operator=(const ParserPosition &) = default;
 
   bool isValid() const { return LS.isValid(); }
 };

lib/Sema/LookupVisibleDecls.cpp

@@ -70,8 +70,6 @@ struct LookupState {
         IncludeDerivedRequirements(0), IncludeProtocolExtensionMembers(0) {}
 
 public:
-  LookupState(const LookupState &) = default;
-
   static LookupState makeQualified() {
     LookupState Result;
     Result.IsQualified = 1;