diff options
| author | Maxime Chevalier-Boisvert <maxime.chevalierboisvert@shopify.com> | 2022-09-20 19:30:10 -0400 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-09-20 19:30:10 -0400 |
| commit | d3733c2ba5207d29ef5b52a93672873ae369b52c (patch) | |
| tree | 4a83e3d54bd7991518f33e0e55fde03fe3ec80fd /yjit | |
| parent | a8dc49b4d57e0a2d07691176f43e4b7f71874fa7 (diff) | |
Guard `--yjit-stats` behind `#[cfg(feature = "stats")]` (#6409)
* Guard --yjit-stats behind #[cfg(feature = "stats")]
* Only ask for --yjit-stats with dev builds on cirrus CI
* Revert "Only ask for --yjit-stats with dev builds on cirrus CI"
This reverts commit cfb5ddfa4b9394ca240447eee02637788435b02a.
* Make it so the --yjit-stats option works for non-release builds
* Revert accidental changes
Notes
Notes:
Merged-By: maximecb <maximecb@ruby-lang.org>
Diffstat (limited to 'yjit')
| -rw-r--r-- | yjit/src/options.rs | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/yjit/src/options.rs b/yjit/src/options.rs index cad7bf332a..e588876173 100644 --- a/yjit/src/options.rs +++ b/yjit/src/options.rs @@ -151,8 +151,10 @@ 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."); @@ -161,6 +163,7 @@ pub fn parse_option(str_ptr: *const std::os::raw::c_char) -> Option<()> { 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 }, |