diff options
| author | Takashi Kokubun <takashi.kokubun@shopify.com> | 2025-10-31 12:47:55 -0700 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2025-10-31 12:47:55 -0700 |
| commit | ab01fcc5123205cdff6e566c2b686e7ab3ed383f (patch) | |
| tree | b7e6cc9db4f7e2665f1ad6dd7b8f4ba12e6f22d3 | |
| parent | 980e18496e1aafc642b199d24c81ab4a8afb3abb (diff) | |
ZJIT: Let AssemblerPanicHook write into stderr (#15019)
| -rw-r--r-- | zjit/src/backend/lir.rs | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/zjit/src/backend/lir.rs b/zjit/src/backend/lir.rs index a90e80bf41..584251de80 100644 --- a/zjit/src/backend/lir.rs +++ b/zjit/src/backend/lir.rs @@ -2290,7 +2290,7 @@ pub struct AssemblerPanicHook { impl AssemblerPanicHook { /// Maximum number of lines [`Self::dump_asm`] is allowed to dump by default. /// When --zjit-dump-lir is given, this limit is ignored. - const MAX_DUMP_LINES: usize = 40; + const MAX_DUMP_LINES: usize = 10; /// Install a panic hook to dump Assembler with insn_idx on dev builds. /// This returns shared references to the previous hook and insn_idx. @@ -2340,12 +2340,12 @@ impl AssemblerPanicHook { (insn_idx.saturating_sub(Self::MAX_DUMP_LINES / 2), insn_idx.saturating_add(Self::MAX_DUMP_LINES / 2)) }; - println!("Failed to compile LIR at insn_idx={insn_idx}:"); + eprintln!("Failed to compile LIR at insn_idx={insn_idx}:"); for (idx, line) in lines.iter().enumerate().filter(|(idx, _)| (min_idx..=max_idx).contains(idx)) { if idx == insn_idx && line.starts_with(" ") { - println!("{BOLD_BEGIN}=>{}{BOLD_END}", &line[" ".len()..]); + eprintln!("{BOLD_BEGIN}=>{}{BOLD_END}", &line[" ".len()..]); } else { - println!("{line}"); + eprintln!("{line}"); } } } |
