1berry/ruby
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

YJIT: Skip dump-disasm if it fails to create a file (#8968)

Browse Source
This commit is contained in:
Takashi Kokubun 2023-11-21 07:28:45 -08:00 committed by GitHub
parent 7e7e2dde24
commit 95d4a52b3a
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 9 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
yjit/src/options.rs
View File

@ -1,4 +1,4 @@
use std::{ffi::{CStr, CString}, ptr::null}; use std::{ffi::{CStr, CString}, ptr::null, fs::File};
use crate::backend::current::TEMP_REGS; use crate::backend::current::TEMP_REGS;
use std::os::raw::{c_char, c_int, c_uint}; use std::os::raw::{c_char, c_int, c_uint};
@ -230,11 +230,15 @@ pub fn parse_option(str_ptr: *const std::os::raw::c_char) -> Option<()> {
("dump-disasm", _) => match opt_val { ("dump-disasm", _) => match opt_val {
"" => unsafe { OPTIONS.dump_disasm = Some(DumpDisasm::Stdout) }, "" => unsafe { OPTIONS.dump_disasm = Some(DumpDisasm::Stdout) },
directory => { directory => {
let pid = std::process::id(); let path = format!("{directory}/yjit_{}.log", std::process::id());
let path = format!("{directory}/yjit_{pid}.log"); match File::options().create(true).append(true).open(&path) {
Ok(_) => {
eprintln!("YJIT disasm dump: {path}"); eprintln!("YJIT disasm dump: {path}");
unsafe { OPTIONS.dump_disasm = Some(DumpDisasm::File(path)) } unsafe { OPTIONS.dump_disasm = Some(DumpDisasm::File(path)) }
} }
Err(err) => eprintln!("Failed to create {path}: {err}"),
}
}
}, },
("dump-iseq-disasm", _) => unsafe { ("dump-iseq-disasm", _) => unsafe {