Create the previous dep graph index on a background thread

Author: Zoxc

compiler/rustc_data_structures/src/marker.rs

@@ -71,6 +71,7 @@ macro_rules! impl_dyn_send {
 }
 
 impl_dyn_send!(
+    [std::thread::JoinHandle<T> where T]
     [std::sync::atomic::AtomicPtr<T> where T]
     [std::sync::Mutex<T> where T: ?Sized+ DynSend]
     [std::sync::mpsc::Sender<T> where T: DynSend]
@@ -154,6 +155,7 @@ macro_rules! impl_dyn_sync {
 }
 
 impl_dyn_sync!(
+    [std::thread::JoinHandle<T> where T]
     [std::sync::atomic::AtomicPtr<T> where T]
     [std::sync::OnceLock<T> where T: DynSend + DynSync]
     [std::sync::Mutex<T> where T: ?Sized + DynSend]

compiler/rustc_incremental/src/persist/load.rs

@@ -35,9 +35,9 @@ pub enum LoadResult<T> {
     LoadDepGraph(PathBuf, std::io::Error),
 }
 
-impl<T: Default> LoadResult<T> {
+impl<T> LoadResult<T> {
     /// Accesses the data returned in [`LoadResult::Ok`].
-    pub fn open(self, sess: &Session) -> T {
+    pub fn open(self, sess: &Session, fallback: impl FnOnce() -> T) -> T {
         // Check for errors when using `-Zassert-incremental-state`
         match (sess.opts.assert_incr_state, &self) {
             (Some(IncrementalStateAssertion::NotLoaded), LoadResult::Ok { .. }) => {
@@ -55,14 +55,14 @@ impl<T: Default> LoadResult<T> {
         match self {
             LoadResult::LoadDepGraph(path, err) => {
                 sess.dcx().emit_warn(errors::LoadDepGraph { path, err });
-                Default::default()
+                fallback()
             }
             LoadResult::DataOutOfDate => {
                 if let Err(err) = delete_all_session_dir_contents(sess) {
                     sess.dcx()
                         .emit_err(errors::DeleteIncompatible { path: dep_graph_path(sess), err });
                 }
-                Default::default()
+                fallback()
             }
             LoadResult::Ok { data } => data,
         }
@@ -91,12 +91,16 @@ fn delete_dirty_work_product(sess: &Session, swp: SerializedWorkProduct) {
     work_product::delete_workproduct_files(sess, &swp.work_product);
 }
 
-fn load_dep_graph(sess: &Session) -> LoadResult<(Arc<SerializedDepGraph>, WorkProductMap)> {
+fn load_dep_graph(
+    sess: &Session,
+) -> LoadResult<(Arc<SerializedDepGraph<DepsType>>, WorkProductMap)> {
     let prof = sess.prof.clone();
 
     if sess.opts.incremental.is_none() {
         // No incremental compilation.
-        return LoadResult::Ok { data: Default::default() };
+        return LoadResult::Ok {
+            data: (Arc::new(SerializedDepGraph::empty(sess)), Default::default()),
+        };
     }
 
     let _timer = sess.prof.generic_activity("incr_comp_prepare_load_dep_graph");
@@ -171,7 +175,7 @@ fn load_dep_graph(sess: &Session) -> LoadResult<(Arc<SerializedDepGraph>, WorkPr
                 return LoadResult::DataOutOfDate;
             }
 
-            let dep_graph = SerializedDepGraph::decode::<DepsType>(&mut decoder);
+            let dep_graph = SerializedDepGraph::decode(&mut decoder, sess);
 
             LoadResult::Ok { data: (dep_graph, prev_work_products) }
         }
@@ -228,7 +232,8 @@ pub fn setup_dep_graph(
     }
 
     res.and_then(|result| {
-        let (prev_graph, prev_work_products) = result.open(sess);
+        let (prev_graph, prev_work_products) =
+            result.open(sess, || (Arc::new(SerializedDepGraph::empty(sess)), Default::default()));
         build_dep_graph(sess, prev_graph, prev_work_products)
     })
     .unwrap_or_else(DepGraph::new_disabled)

compiler/rustc_incremental/src/persist/save.rs

@@ -4,7 +4,7 @@ use std::sync::Arc;
 use rustc_data_structures::fx::FxIndexMap;
 use rustc_data_structures::sync::join;
 use rustc_middle::dep_graph::{
-    DepGraph, SerializedDepGraph, WorkProduct, WorkProductId, WorkProductMap,
+    DepGraph, DepsType, SerializedDepGraph, WorkProduct, WorkProductId, WorkProductMap,
 };
 use rustc_middle::ty::TyCtxt;
 use rustc_serialize::Encodable as RustcEncodable;
@@ -144,7 +144,7 @@ fn encode_query_cache(tcx: TyCtxt<'_>, encoder: FileEncoder) -> FileEncodeResult
 /// and moves it to the permanent dep-graph path
 pub(crate) fn build_dep_graph(
     sess: &Session,
-    prev_graph: Arc<SerializedDepGraph>,
+    prev_graph: Arc<SerializedDepGraph<DepsType>>,
     prev_work_products: WorkProductMap,
 ) -> Option<DepGraph> {
     if sess.opts.incremental.is_none() {

compiler/rustc_query_system/src/dep_graph/graph.rs

@@ -91,7 +91,7 @@ pub(crate) struct DepGraphData<D: Deps> {
 
     /// The dep-graph from the previous compilation session. It contains all
     /// nodes and edges as well as all fingerprints of nodes that have them.
-    previous: Arc<SerializedDepGraph>,
+    previous: Arc<SerializedDepGraph<D>>,
 
     colors: DepNodeColorMap,
 
@@ -121,7 +121,7 @@ where
 impl<D: Deps> DepGraph<D> {
     pub fn new(
         session: &Session,
-        prev_graph: Arc<SerializedDepGraph>,
+        prev_graph: Arc<SerializedDepGraph<D>>,
         prev_work_products: WorkProductMap,
         encoder: FileEncoder,
     ) -> DepGraph<D> {
@@ -1192,7 +1192,7 @@ impl<D: Deps> CurrentDepGraph<D> {
         session: &Session,
         prev_graph_node_count: usize,
         encoder: FileEncoder,
-        previous: Arc<SerializedDepGraph>,
+        previous: Arc<SerializedDepGraph<D>>,
     ) -> Self {
         let mut stable_hasher = StableHasher::new();
         previous.session_count().hash(&mut stable_hasher);
@@ -1281,7 +1281,7 @@ impl<D: Deps> CurrentDepGraph<D> {
     #[inline]
     fn debug_assert_not_in_new_nodes(
         &self,
-        prev_graph: &SerializedDepGraph,
+        prev_graph: &SerializedDepGraph<D>,
         prev_index: SerializedDepNodeIndex,
     ) {
         if let Some(ref nodes_in_current_session) = self.nodes_in_current_session {

compiler/rustc_query_system/src/dep_graph/mod.rs

@@ -12,7 +12,7 @@ pub(crate) use graph::DepGraphData;
 pub use graph::{DepGraph, DepNodeIndex, TaskDepsRef, WorkProduct, WorkProductMap, hash_result};
 pub use query::DepGraphQuery;
 use rustc_data_structures::profiling::SelfProfilerRef;
-use rustc_data_structures::sync::DynSync;
+use rustc_data_structures::sync::{DynSend, DynSync};
 use rustc_session::Session;
 pub use serialized::{SerializedDepGraph, SerializedDepNodeIndex};
 use tracing::instrument;
@@ -92,7 +92,7 @@ pub trait DepContext: Copy {
     fn with_reduced_queries<T>(self, _: impl FnOnce() -> T) -> T;
 }
 
-pub trait Deps: DynSync {
+pub trait Deps: DynSend + DynSync + Send + Sync + 'static {
     /// Execute the operation with provided dependencies.
     fn with_deps<OP, R>(deps: TaskDepsRef<'_>, op: OP) -> R
     where