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

YJIT: print warning when disasm options used without a dev build (#9744)

Browse Source 
* YJIT: print warning when disasm options used without a dev build

I was confused for a few minutes the other way then
--yjit-dump-disasm printed nothing, so I figured this would be
useful for end-users (and future me).

* Fix lone extraneous space
This commit is contained in:
Maxime Chevalier-Boisvert 2024-01-29 16:17:12 -05:00 committed by GitHub
parent 32bbf47500
commit bc10b958d2
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 20 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
yjit/src/options.rs
View File

@ -228,7 +228,12 @@ pub fn parse_option(str_ptr: *const std::os::raw::c_char) -> Option<()> {
_ => return None,
},
("dump-disasm", _) => match opt_val {
("dump-disasm", _) => {
if !cfg!(feature = "disasm") {
eprintln!("WARNING: the {} option is only available when YJIT is built in dev mode, i.e. ./configure --enable-yjit=dev", opt_name);
}
match opt_val {
"" => unsafe { OPTIONS.dump_disasm = Some(DumpDisasm::Stdout) },
directory => {
let path = format!("{directory}/yjit_{}.log", std::process::id());
@ -240,9 +245,14 @@ pub fn parse_option(str_ptr: *const std::os::raw::c_char) -> Option<()> {
Err(err) => eprintln!("Failed to create {path}: {err}"),
}
}
}
},
("dump-iseq-disasm", _) => unsafe {
if !cfg!(feature = "disasm") {
eprintln!("WARNING: the {} option is only available when YJIT is built in dev mode, i.e. ./configure --enable-yjit=dev", opt_name);
}
OPTIONS.dump_iseq_disasm = Some(opt_val.to_string());
},