summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTakashi Kokubun <takashi.kokubun@shopify.com>2025-10-31 12:47:55 -0700
committerGitHub <noreply@github.com>2025-10-31 12:47:55 -0700
commitab01fcc5123205cdff6e566c2b686e7ab3ed383f (patch)
treeb7e6cc9db4f7e2665f1ad6dd7b8f4ba12e6f22d3
parent980e18496e1aafc642b199d24c81ab4a8afb3abb (diff)
ZJIT: Let AssemblerPanicHook write into stderr (#15019)
-rw-r--r--zjit/src/backend/lir.rs8
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}");
}
}
}