[nfc] localize getAnyTypeDecl

After review, Slava and I decided it's not a good idea to provide a
`TypeBase::getAnyTypeDecl` method, as it is not typically what people
generally should use, as it doesn't handle all cases (type parameters
with no associated decl, for example).

Author: kavon

include/swift/AST/Type.h

@@ -541,7 +541,6 @@ class CanType : public Type {
   CanType getNominalParent() const; // in Types.h
   NominalTypeDecl *getAnyNominal() const;
   GenericTypeDecl *getAnyGeneric() const;
-  TypeDecl *getAnyTypeDecl() const;
 
   bool isForeignReferenceType(); // in Types.h
 

include/swift/AST/Types.h

@@ -1233,11 +1233,6 @@ class alignas(1 << TypeAlignInBits) TypeBase
   /// declaration.
   GenericTypeDecl *getAnyGeneric();
 
-  // If this a GenericType, ModuleType, or GenericTypeParamType, return the
-  // type declaration.
-  // NOTE: Will not attempt to find an AssociatedTypeDecl.
-  TypeDecl *getAnyTypeDecl();
-
   /// removeArgumentLabels -  Retrieve a version of this type with all
   /// argument labels removed.
   Type removeArgumentLabels(unsigned numArgumentLabels);
@@ -7426,10 +7421,6 @@ inline GenericTypeDecl *TypeBase::getAnyGeneric() {
   return getCanonicalType().getAnyGeneric();
 }
 
-inline TypeDecl *TypeBase::getAnyTypeDecl() {
-  return getCanonicalType().getAnyTypeDecl();
-}
-
 inline bool TypeBase::isBuiltinIntegerType(unsigned n) {
   if (auto intTy = dyn_cast<BuiltinIntegerType>(getCanonicalType()))
     return intTy->getWidth().isFixedWidth()

lib/AST/ASTPrinter.cpp

@@ -3336,6 +3336,14 @@ static bool hasInverseCopyable(
     Decl *decl,
     std::function<bool(InverseMarking const&)> isRelevantInverse) {
 
+  auto getTypeDecl = [](Type type) -> TypeDecl* {
+    if (auto genericTy = type->getAnyGeneric())
+      return genericTy;
+    if (auto gtpt = dyn_cast<GenericTypeParamType>(type))
+      return gtpt->getDecl();
+    return nullptr;
+  };
+
   if (auto *extension = dyn_cast<ExtensionDecl>(decl)) {
     if (auto *nominal = extension->getSelfNominalTypeDecl())
       if (isRelevantInverse(nominal->getNoncopyableMarking()))
@@ -3361,7 +3369,7 @@ static bool hasInverseCopyable(
     // Check for noncopyable types in the types of this declaration.
     if (Type type = value->getInterfaceType()) {
       bool hasNoncopyable = type.findIf([&](Type type) {
-        if (auto typeDecl = type->getAnyTypeDecl())
+        if (auto *typeDecl = getTypeDecl(type))
           if (isRelevantInverse(typeDecl->getNoncopyableMarking()))
             return true;
 

lib/AST/Type.cpp

@@ -91,17 +91,6 @@ GenericTypeDecl *CanType::getAnyGeneric() const {
   return nullptr;
 }
 
-TypeDecl *CanType::getAnyTypeDecl() const {
-  // NOTE: there is no simple way to determine if it's an AssociatedTypeDecl.
-  if (auto genericTy = getAnyGeneric())
-    return genericTy;
-  if (auto gtpt = dyn_cast<GenericTypeParamType>(*this))
-    return gtpt->getDecl();
-  if (auto module = dyn_cast<ModuleType>(*this))
-    return module->getModule();
-  return nullptr;
-}
-
 //===----------------------------------------------------------------------===//
 // Various Type Methods.
 //===----------------------------------------------------------------------===//