diff options
| author | Michael Goulet <michael@errs.io> | 2025-06-10 21:09:35 +0000 |
|---|---|---|
| committer | Michael Goulet <michael@errs.io> | 2025-06-25 15:45:36 +0000 |
| commit | d0321192532f3b32e187e43a183c92c31fcfe83a (patch) | |
| tree | 037a55c59385fd8fd46380f7bccfb2c9b6e3ce18 | |
| parent | 2801f9aaf9b7580d9b230b532b0700709857cc88 (diff) | |
| download | rust-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.rs | 11 |
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) } |
