Merge pull request #24272 from compnerd/not-equal

AST: inline the definition for `AnyValue::Holder<Type>::equals`

Author: compnerd

include/swift/AST/TypeCheckRequests.h

@@ -550,7 +550,11 @@ class PropertyDelegateBackingPropertyInfoRequest :
 // Allow AnyValue to compare two Type values, even though Type doesn't
 // support ==.
 template<>
-bool AnyValue::Holder<Type>::equals(const HolderBase &other) const;
+inline bool AnyValue::Holder<Type>::equals(const HolderBase &other) const {
+  assert(typeID == other.typeID && "Caller should match type IDs");
+  return value.getPointer() ==
+      static_cast<const Holder<Type> &>(other).value.getPointer();
+}
 
 void simple_display(llvm::raw_ostream &out, const Type &type);
 

lib/AST/TypeCheckRequests.cpp

@@ -646,14 +646,6 @@ void swift::simple_display(
   out << " }";
 }
 
-
-template<>
-bool swift::AnyValue::Holder<Type>::equals(const HolderBase &other) const {
-  assert(typeID == other.typeID && "Caller should match type IDs");
-  return value.getPointer() ==
-  static_cast<const Holder<Type> &>(other).value.getPointer();
-}
-
 void swift::simple_display(llvm::raw_ostream &out, const Type &type) {
   if (type)
     type.print(out);