Cache result of check_path_modifications

Author: Kobzol

src/bootstrap/src/core/config/config.rs

@@ -10,7 +10,7 @@ use std::io::IsTerminal;
 use std::path::{Path, PathBuf, absolute};
 use std::process::Command;
 use std::str::FromStr;
-use std::sync::OnceLock;
+use std::sync::{LazyLock, Mutex, OnceLock};
 use std::{cmp, env, fs};
 
 use build_helper::ci::CiEnv;
@@ -3130,17 +3130,34 @@ impl Config {
     }
 
     /// Returns true if any of the `paths` have been modified locally.
-    pub fn has_changes_from_upstream(&self, paths: &[&str]) -> bool {
+    pub fn has_changes_from_upstream(&self, paths: &[&'static str]) -> bool {
         match self.check_path_modifications(paths) {
             PathFreshness::LastModifiedUpstream { .. } => false,
             PathFreshness::HasLocalModifications { .. } | PathFreshness::MissingUpstream => true,
         }
     }
 
     /// Checks whether any of the given paths have been modified w.r.t. upstream.
-    pub fn check_path_modifications(&self, paths: &[&str]) -> PathFreshness {
-        check_path_modifications(Some(&self.src), &self.git_config(), paths, CiEnv::current())
+    pub fn check_path_modifications(&self, paths: &[&'static str]) -> PathFreshness {
+        // Checking path modifications through git can be relatively expensive (>100ms).
+        // We do not assume that the sources would change during bootstrap's execution,
+        // so we can cache the results here.
+        static MODIFICATION_CACHE: LazyLock<Mutex<HashMap<Vec<&'static str>, PathFreshness>>> =
+            LazyLock::new(|| Mutex::new(Default::default()));
+        MODIFICATION_CACHE
+            .lock()
             .unwrap()
+            .entry(paths.to_vec())
+            .or_insert_with(|| {
+                check_path_modifications(
+                    Some(&self.src),
+                    &self.git_config(),
+                    paths,
+                    CiEnv::current(),
+                )
+                .unwrap()
+            })
+            .clone()
     }
 }
 

src/build_helper/src/git.rs

@@ -32,7 +32,7 @@ pub fn output_result(cmd: &mut Command) -> Result<String, String> {
 
 /// Represents the result of checking whether a set of paths
 /// have been modified locally or not.
-#[derive(PartialEq, Debug)]
+#[derive(PartialEq, Debug, Clone)]
 pub enum PathFreshness {
     /// Artifacts should be downloaded from this upstream commit,
     /// there are no local modifications.