feat: re-add old debug_info_by_pid field for old report backend parsing

Author: GuillaumeLagrange

crates/runner-shared/src/metadata.rs

@@ -28,6 +28,11 @@ pub struct PerfMetadata {
     #[deprecated(note = "Use ExecutionTimestamps in the 'artifacts' module instead")]
     pub markers: Vec<MarkerType>,
 
+    /// Kept for backward compatibility, was used before deduplication of debug info entries.
+    #[serde(default, skip_serializing_if = "HashMap::is_empty")]
+    #[deprecated(note = "Use 'debug_info' + 'debug_info_pid_mappings_by_pid' instead")]
+    pub debug_info_by_pid: HashMap<i32, Vec<ModuleDebugInfo>>,
+
     /// Deduplicated debug info entries, keyed by semantic key
     #[serde(default, skip_serializing_if = "HashMap::is_empty")]
     pub debug_info: HashMap<String, ModuleDebugInfo>,

src/executor/wall_time/perf/mod.rs

@@ -502,6 +502,7 @@ impl BenchmarkData {
             debug_info_pid_mappings_by_pid,
             unwind_data_pid_mappings_by_pid,
             symbol_pid_mappings_by_pid,
+            debug_info_by_pid: Default::default(), // No longer used
         };
         metadata.save_to(&path).unwrap();