bjoernager/rust
bjoernager/rust
1
Fork 0
Code Activity

Stacker now handles miri using a noop impl itself

Browse source
This commit is contained in:
bjorn3 2025-02-13 17:33:14 +00:00
parent a18bd8acfc
commit 7b74920388
2 changed files with 4 additions and 16 deletions
Download patch file Download diff file Expand all files Collapse all files

12
compiler/rustc_data_structures/src/stack.rs
View file

@ -17,18 +17,6 @@ const STACK_PER_RECURSION: usize = 16 * 1024 * 1024; // 16MB
///
/// Should not be sprinkled around carelessly, as it causes a little bit of overhead.
#[inline]
#[cfg(not(miri))]
pub fn ensure_sufficient_stack<R>(f: impl FnOnce() -> R) -> R {
stacker::maybe_grow(RED_ZONE, STACK_PER_RECURSION, f)
}
/// Grows the stack on demand to prevent stack overflow. Call this in strategic locations
/// to "break up" recursive calls. E.g. almost any call to `visit_expr` or equivalent can benefit
/// from this.
///
/// Should not be sprinkled around carelessly, as it causes a little bit of overhead.
#[cfg(miri)]
#[inline]
pub fn ensure_sufficient_stack<R>(f: impl FnOnce() -> R) -> R {
f()
}