Revert "[mypy] perf"

hauntsaninja · hauntsaninja · commit 273240aee40e · 2025-08-17T12:28:29.000-07:00
This reverts commit 665f5ad.
diff --git a/mypy/types.py b/mypy/types.py
@@ -1471,7 +1471,7 @@ def accept(self, visitor: TypeVisitor[T]) -> T:
 
     def __hash__(self) -> int:
         if self._hash == -1:
-            self._hash = hash((self.type, self.args, self.last_known_value))
+            self._hash = hash((self.type, self.args, self.last_known_value, self.extra_attrs))
         return self._hash
 
     def __eq__(self, other: object) -> bool:
@@ -2895,7 +2895,6 @@ class UnionType(ProperType):
 
     __slots__ = (
         "items",
-        "_frozen_items",
         "is_evaluated",
         "uses_pep604_syntax",
         "original_str_expr",
@@ -2915,7 +2914,6 @@ def __init__(
         # We must keep this false to avoid crashes during semantic analysis.
         # TODO: maybe switch this to True during type-checking pass?
         self.items = flatten_nested_unions(items, handle_type_alias_type=False)
-        self._frozen_items = frozenset(self.items)
         # is_evaluated should be set to false for type comments and string literals
         self.is_evaluated = is_evaluated
         # uses_pep604_syntax is True if Union uses OR syntax (X | Y)
@@ -2933,14 +2931,14 @@ def can_be_false_default(self) -> bool:
         return any(item.can_be_false for item in self.items)
 
     def __hash__(self) -> int:
-        return hash(self._frozen_items)
+        return hash(frozenset(self.items))
 
     def __eq__(self, other: object) -> bool:
         if not isinstance(other, UnionType):
             return NotImplemented
         if self is other:
             return True
-        return self._frozen_items == other._frozen_items
+        return frozenset(self.items) == frozenset(other.items)
 
     @overload
     @staticmethod
