Skip to content

Commit

Permalink
Merge pull request #19004 from teoxoy/patch-1
Browse files Browse the repository at this point in the history
increase `AUTODEREF_RECURSION_LIMIT` to 20
  • Loading branch information
ChayimFriedman2 authored Jan 22, 2025
2 parents 1f23156 + 90533a0 commit 582af7e
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions crates/hir-ty/src/autoderef.rs
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ use crate::{
TraitEnvironment, Ty, TyBuilder, TyKind,
};

static AUTODEREF_RECURSION_LIMIT: Limit = Limit::new(10);
static AUTODEREF_RECURSION_LIMIT: Limit = Limit::new(20);

#[derive(Debug)]
pub(crate) enum AutoderefKind {
Expand Down Expand Up @@ -49,7 +49,7 @@ pub fn autoderef(
// If the deref chain contains a cycle (e.g. `A` derefs to `B` and `B` derefs to `A`), we
// would revisit some already visited types. Stop here to avoid duplication.
//
// XXX: The recursion limit for `Autoderef` is currently 10, so `Vec::contains()` shouldn't
// XXX: The recursion limit for `Autoderef` is currently 20, so `Vec::contains()` shouldn't
// be too expensive. Replace this duplicate check with `FxHashSet` if it proves to be more
// performant.
if v.contains(&resolved) {
Expand Down

0 comments on commit 582af7e

Please sign in to comment.