about summary refs log tree commit diff
diff options
context:
space:
mode:
authorMichael Goulet <michael@errs.io>2025-06-10 21:09:35 +0000
committerMichael Goulet <michael@errs.io>2025-06-25 15:45:36 +0000
commitd0321192532f3b32e187e43a183c92c31fcfe83a (patch)
tree037a55c59385fd8fd46380f7bccfb2c9b6e3ce18
parent2801f9aaf9b7580d9b230b532b0700709857cc88 (diff)
downloadrust-d0321192532f3b32e187e43a183c92c31fcfe83a.tar.gz
rust-d0321192532f3b32e187e43a183c92c31fcfe83a.zip
Don't fold in Instantiate when there's nothing to fold
-rw-r--r--compiler/rustc_type_ir/src/binder.rs11
1 files changed, 11 insertions, 0 deletions
diff --git a/compiler/rustc_type_ir/src/binder.rs b/compiler/rustc_type_ir/src/binder.rs
index 927a2ce84ea..a7b915c4845 100644
--- a/compiler/rustc_type_ir/src/binder.rs
+++ b/compiler/rustc_type_ir/src/binder.rs
@@ -622,6 +622,17 @@ impl<I: Interner, T: TypeFoldable<I>> ty::EarlyBinder<I, T> {
     where
         A: SliceLike<Item = I::GenericArg>,
     {
+        // Nothing to fold, so let's avoid visiting things and possibly re-hashing/equating
+        // them when interning. Perf testing found this to be a modest improvement.
+        // See: <https://github.com/rust-lang/rust/pull/142317>
+        if args.is_empty() {
+            assert!(
+                !self.value.has_param(),
+                "{:?} has parameters, but no args were provided in instantiate",
+                self.value,
+            );
+            return self.value;
+        }
         let mut folder = ArgFolder { cx, args: args.as_slice(), binders_passed: 0 };
         self.value.fold_with(&mut folder)
     }