--wip-- [skip ci]

Author: GuillaumeLagrange

src/executor/wall_time/perf/save_artifacts.rs

@@ -20,6 +20,46 @@ pub struct SavedArtifacts {
     pub key_to_path: HashMap<String, PathBuf>,
 }
 
+/// Save all artifacts (symbols, debug info, unwind data) from mounted modules and JIT data.
+pub fn save_artifacts(
+    profile_folder: &Path,
+    mounted_modules_by_path: &HashMap<PathBuf, MountedModule>,
+    jit_unwind_data_by_pid: &HashMap<pid_t, Vec<(UnwindData, ProcessUnwindData)>>,
+) -> SavedArtifacts {
+    let mut path_to_key = HashMap::<PathBuf, String>::new();
+
+    register_paths(&mut path_to_key, mounted_modules_by_path);
+
+    let symbol_pid_mappings_by_pid =
+        save_symbols(profile_folder, mounted_modules_by_path, &path_to_key);
+
+    let (debug_info, debug_info_pid_mappings_by_pid) =
+        save_debug_info(mounted_modules_by_path, &mut path_to_key);
+
+    let unwind_data_pid_mappings_by_pid = save_unwind_data(
+        profile_folder,
+        mounted_modules_by_path,
+        jit_unwind_data_by_pid,
+        &mut path_to_key,
+    );
+
+    let ignored_modules = collect_ignored_modules(mounted_modules_by_path);
+
+    let key_to_path = path_to_key
+        .into_iter()
+        .map(|(path, key)| (key, path))
+        .collect();
+
+    SavedArtifacts {
+        symbol_pid_mappings_by_pid,
+        debug_info,
+        debug_info_pid_mappings_by_pid,
+        unwind_data_pid_mappings_by_pid,
+        ignored_modules,
+        key_to_path,
+    }
+}
+
 /// Register a path in the map if absent, assigning a new unique key.
 /// Returns the assigned key.
 fn get_or_insert_key(path_to_key: &mut HashMap<PathBuf, String>, path: &Path) -> String {
@@ -238,43 +278,3 @@ fn collect_ignored_modules(
 
     to_ignore
 }
-
-/// Save all artifacts (symbols, debug info, unwind data) from mounted modules and JIT data.
-pub fn save_artifacts(
-    profile_folder: &Path,
-    mounted_modules_by_path: &HashMap<PathBuf, MountedModule>,
-    jit_unwind_data_by_pid: &HashMap<pid_t, Vec<(UnwindData, ProcessUnwindData)>>,
-) -> SavedArtifacts {
-    let mut path_to_key = HashMap::<PathBuf, String>::new();
-
-    register_paths(&mut path_to_key, mounted_modules_by_path);
-
-    let symbol_pid_mappings_by_pid =
-        save_symbols(profile_folder, mounted_modules_by_path, &path_to_key);
-
-    let (debug_info, debug_info_pid_mappings_by_pid) =
-        save_debug_info(mounted_modules_by_path, &mut path_to_key);
-
-    let unwind_data_pid_mappings_by_pid = save_unwind_data(
-        profile_folder,
-        mounted_modules_by_path,
-        jit_unwind_data_by_pid,
-        &mut path_to_key,
-    );
-
-    let ignored_modules = collect_ignored_modules(mounted_modules_by_path);
-
-    let key_to_path = path_to_key
-        .into_iter()
-        .map(|(path, key)| (key, path))
-        .collect();
-
-    SavedArtifacts {
-        symbol_pid_mappings_by_pid,
-        debug_info,
-        debug_info_pid_mappings_by_pid,
-        unwind_data_pid_mappings_by_pid,
-        ignored_modules,
-        key_to_path,
-    }
-}