feat: add support for analysis mode

Author: not-matthias

crates/codspeed/src/codspeed.rs

@@ -1,4 +1,4 @@
-use crate::measurement;
+use crate::{instrument_hooks::InstrumentHooks, measurement};
 use colored::Colorize;
 use std::ffi::CString;
 
@@ -19,7 +19,8 @@ pub struct CodSpeed {
 
 impl CodSpeed {
     pub fn new() -> Self {
-        let is_instrumented = measurement::is_instrumented();
+        let hooks_instance = InstrumentHooks::instance();
+        let is_instrumented = hooks_instance.is_instrumented();
         if !is_instrumented {
             println!(
                 "{} codspeed is enabled, but no performance measurement will be made since it's running in an unknown environment.",
@@ -46,12 +47,16 @@ impl CodSpeed {
     #[inline(always)]
     pub fn start_benchmark(&mut self, name: &str) {
         self.current_benchmark = CString::new(name).expect("CString::new failed");
+        let _ = InstrumentHooks::instance().start_benchmark();
         measurement::start();
     }
 
     #[inline(always)]
     pub fn end_benchmark(&mut self) {
         measurement::stop(&self.current_benchmark);
+        let _ = InstrumentHooks::instance().stop_benchmark();
+        let _ = InstrumentHooks::instance()
+            .set_executed_benchmark(&self.current_benchmark.to_string_lossy());
         self.benchmarked
             .push(self.current_benchmark.to_str().unwrap().to_string());
         let action_str = if self.is_instrumented {

crates/codspeed/src/instrument_hooks/mod.rs

@@ -34,6 +34,7 @@ mod linux_impl {
                 instance
                     .set_integration("codspeed-rust", env!("CARGO_PKG_VERSION"))
                     .expect("Failed to set integration");
+                InstrumentHooks::disable_callgrind_markers();
                 instance
             })
         }
@@ -129,6 +130,15 @@ mod linux_impl {
                 ffi::instrument_hooks_current_timestamp()
             }
         }
+
+        pub fn disable_callgrind_markers() {
+            unsafe {
+                ffi::instrument_hooks_set_feature(
+                    ffi::instrument_hooks_feature_t_FEATURE_DISABLE_CALLGRIND_MARKERS,
+                    true,
+                )
+            };
+        }
     }
 
     impl Drop for InstrumentHooks {
@@ -175,6 +185,8 @@ mod other_impl {
         pub fn current_timestamp() -> u64 {
             0
         }
+
+        pub fn disable_callgrind_markers() {}
     }
 }
 

crates/codspeed/src/measurement.rs

@@ -2,17 +2,6 @@ use std::ffi::CString;
 
 use crate::request::{send_client_request, ClientRequest, Value};
 
-#[inline(always)]
-pub fn is_instrumented() -> bool {
-    let valgrind_depth = unsafe {
-        send_client_request(
-            0,
-            &[ClientRequest::RunningOnValgrind as Value, 0, 0, 0, 0, 0],
-        )
-    };
-    valgrind_depth > 0
-}
-
 #[inline(always)]
 pub fn set_metadata() {
     let full_metadata = CString::new(format!(

crates/codspeed/src/request/mod.rs

@@ -3,7 +3,6 @@ const CG_BASE: u32 = ((b'C' as u32) << 24) + ((b'T' as u32) << 16);
 #[allow(non_camel_case_types)]
 #[repr(u32)]
 pub enum ClientRequest {
-    RunningOnValgrind = 0x1001,
     ZeroStatistics = CG_BASE + 1,
     DumpStatisticsAt = CG_BASE + 3,
     StartInstrumentation = CG_BASE + 4,