Add section timing data to JSON --timings output

Author: Kobzol

src/cargo/core/compiler/mod.rs

@@ -105,6 +105,7 @@ use cargo_util_schemas::manifest::TomlDebugInfo;
 use cargo_util_schemas::manifest::TomlTrimPaths;
 use cargo_util_schemas::manifest::TomlTrimPathsValue;
 use rustfix::diagnostics::Applicability;
+pub(crate) use timings::CompilationSection;
 
 const RUSTDOC_CRATE_VERSION_FLAG: &str = "--crate-version";
 

src/cargo/core/compiler/timings.rs

@@ -64,8 +64,9 @@ pub struct Timings<'gctx> {
     cpu_usage: Vec<(f64, f64)>,
 }
 
-/// Section of compilation.
-struct TimingSection {
+/// Section of compilation (e.g. frontend, backend, linking).
+#[derive(Copy, Clone, serde::Serialize)]
+pub struct CompilationSection {
     /// Start of the section, as an offset in seconds from `UnitTime::start`.
     start: f64,
     /// End of the section, as an offset in seconds from `UnitTime::start`.
@@ -89,7 +90,7 @@ struct UnitTime {
     /// Same as `unlocked_units`, but unlocked by rmeta.
     unlocked_rmeta_units: Vec<Unit>,
     /// Individual compilation section durations, gathered from `--json=timings`.
-    sections: HashMap<String, TimingSection>,
+    sections: HashMap<String, CompilationSection>,
 }
 
 impl UnitTime {
@@ -315,6 +316,7 @@ impl<'gctx> Timings<'gctx> {
                 mode: unit_time.unit.mode,
                 duration: unit_time.duration,
                 rmeta_time: unit_time.rmeta_time,
+                sections: unit_time.sections.clone(),
             }
             .to_json_string();
             crate::drop_println!(self.gctx, "{}", msg);
@@ -723,20 +725,20 @@ impl UnitTime {
             .sections
             .insert(
                 name.to_string(),
-                TimingSection {
+                CompilationSection {
                     start: now - self.start,
                     end: None,
                 },
             )
             .is_some()
         {
-            warn!("Compilation section {name} started more than once");
+            warn!("compilation section {name} started more than once");
         }
     }
 
     fn end_section(&mut self, name: &str, now: f64) {
         let Some(section) = self.sections.get_mut(name) else {
-            warn!("Compilation section {name} ended, but it has no start recorded");
+            warn!("compilation section {name} ended, but it has no start recorded");
             return;
         };
         section.end = Some(now - self.start);

src/cargo/util/machine_message.rs

@@ -1,3 +1,4 @@
+use std::collections::HashMap;
 use std::path::{Path, PathBuf};
 
 use cargo_util_schemas::core::PackageIdSpec;
@@ -6,7 +7,7 @@ use serde::ser;
 use serde_json::{json, value::RawValue};
 
 use crate::core::Target;
-use crate::core::compiler::CompileMode;
+use crate::core::compiler::{CompilationSection, CompileMode};
 
 pub trait Message: ser::Serialize {
     fn reason(&self) -> &str;
@@ -87,6 +88,12 @@ impl<'a> Message for BuildScript<'a> {
     }
 }
 
+#[derive(Serialize)]
+pub struct TimingInfoCompilationSection {
+    start: f64,
+    end: f64,
+}
+
 #[derive(Serialize)]
 pub struct TimingInfo<'a> {
     pub package_id: PackageIdSpec,
@@ -95,6 +102,7 @@ pub struct TimingInfo<'a> {
     pub duration: f64,
     #[serde(skip_serializing_if = "Option::is_none")]
     pub rmeta_time: Option<f64>,
+    pub sections: HashMap<String, CompilationSection>,
 }
 
 impl<'a> Message for TimingInfo<'a> {