From 912ea8257a533299d47d71aac8f1b363853493fe Mon Sep 17 00:00:00 2001 From: Takashi Kokubun Date: Sat, 24 Sep 2022 05:17:54 +0900 Subject: YJIT: Support Rust 1.58.1 for --yjit-stats on Arm (#6410) * YJIT: Test Rust 1.58.1 as well on Cirrus * YJIT: Avoid using a Rust 1.60.0 feature * YJIT: Use autoconf to detect support * YJIT: We actually need to run it for checking it properly * YJIT: Try cfg!(target_feature = "lse") * Revert "YJIT: Try cfg!(target_feature = "lse")" This reverts commit 4e2a9ca9a9c83052c23b5e205c91bdf79e88342e. * YJIT: Add --features stats only when it works * Update configure.ac Co-authored-by: Maxime Chevalier-Boisvert --- yjit/src/options.rs | 14 +------------- 1 file changed, 1 insertion(+), 13 deletions(-) (limited to 'yjit') diff --git a/yjit/src/options.rs b/yjit/src/options.rs index e588876173..f73dca67de 100644 --- a/yjit/src/options.rs +++ b/yjit/src/options.rs @@ -151,19 +151,7 @@ pub fn parse_option(str_ptr: *const std::os::raw::c_char) -> Option<()> { ("greedy-versioning", "") => unsafe { OPTIONS.greedy_versioning = true }, ("no-type-prop", "") => unsafe { OPTIONS.no_type_prop = true }, - - ("stats", "") => { - // Insn::IncrCounter uses ldaddal, which works only on ARMv8.1+. - #[cfg(feature = "stats")] - #[cfg(target_arch = "aarch64")] - if !std::arch::is_aarch64_feature_detected!("lse") { - eprintln!("Your processor does not support --yjit-stats. Aborting."); - std::process::exit(1); - } - - unsafe { OPTIONS.gen_stats = true } - }, - + ("stats", "") => unsafe { OPTIONS.gen_stats = true }, ("trace-exits", "") => unsafe { OPTIONS.gen_trace_exits = true; OPTIONS.gen_stats = true }, ("dump-insns", "") => unsafe { OPTIONS.dump_insns = true }, ("verify-ctx", "") => unsafe { OPTIONS.verify_ctx = true }, -- cgit v1.2.3