Skip to content

Commit

Permalink
Cleanups.
Browse files Browse the repository at this point in the history
  • Loading branch information
cjgillot committed Mar 27, 2020
1 parent db5be1f commit 222d010
Show file tree
Hide file tree
Showing 5 changed files with 14 additions and 15 deletions.
10 changes: 5 additions & 5 deletions src/librustc/dep_graph/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -95,6 +95,10 @@ impl<'tcx> DepContext for TyCtxt<'tcx> {
TyCtxt::create_stable_hashing_context(*self)
}

fn debug_dep_tasks(&self) -> bool {
self.sess.opts.debugging_opts.dep_tasks
}

fn try_force_from_dep_node(&self, dep_node: &DepNode) -> bool {
// FIXME: This match is just a workaround for incremental bugs and should
// be removed. https://github.com/rust-lang/rust/issues/62649 is one such
Expand Down Expand Up @@ -181,8 +185,4 @@ fn def_id_corresponds_to_hir_dep_node(tcx: TyCtxt<'_>, def_id: DefId) -> bool {
def_id.index == hir_id.owner.local_def_index
}

impl rustc_query_system::HashStableContext for StableHashingContext<'_> {
fn debug_dep_tasks(&self) -> bool {
self.sess().opts.debugging_opts.dep_tasks
}
}
impl rustc_query_system::HashStableContext for StableHashingContext<'_> {}
10 changes: 5 additions & 5 deletions src/librustc/ty/query/plumbing.rs
Original file line number Diff line number Diff line change
Expand Up @@ -209,14 +209,14 @@ macro_rules! is_eval_always {
}

macro_rules! query_storage {
(<$tcx:tt>[][$K:ty, $V:ty]) => {
([][$K:ty, $V:ty]) => {
<<$K as Key>::CacheSelector as CacheSelector<$K, $V>>::Cache
};
(<$tcx:tt>[storage($ty:ty) $($rest:tt)*][$K:ty, $V:ty]) => {
([storage($ty:ty) $($rest:tt)*][$K:ty, $V:ty]) => {
$ty
};
(<$tcx:tt>[$other:ident $(($($other_args:tt)*))* $(, $($modifiers:tt)*)*][$($args:tt)*]) => {
query_storage!(<$tcx>[$($($modifiers)*)*][$($args)*])
([$other:ident $(($($other_args:tt)*))* $(, $($modifiers:tt)*)*][$($args:tt)*]) => {
query_storage!([$($($modifiers)*)*][$($args)*])
};
}

Expand Down Expand Up @@ -332,7 +332,7 @@ macro_rules! define_queries_inner {
const EVAL_ALWAYS: bool = is_eval_always!([$($modifiers)*]);
const DEP_KIND: dep_graph::DepKind = dep_graph::DepKind::$node;

type Cache = query_storage!(<$tcx>[$($modifiers)*][$K, $V]);
type Cache = query_storage!([$($modifiers)*][$K, $V]);

#[inline(always)]
fn query_state<'a>(tcx: TyCtxt<$tcx>) -> &'a QueryState<TyCtxt<$tcx>, Self::Cache> {
Expand Down
3 changes: 1 addition & 2 deletions src/librustc_query_system/dep_graph/graph.rs
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,6 @@ use super::prev::PreviousDepGraph;
use super::query::DepGraphQuery;
use super::serialized::{SerializedDepGraph, SerializedDepNodeIndex};
use super::{DepContext, DepKind, DepNode, WorkProductId};
use crate::HashStableContext;

#[derive(Clone)]
pub struct DepGraph<K: DepKind> {
Expand Down Expand Up @@ -259,7 +258,7 @@ impl<K: DepKind> DepGraph<K> {
task_deps.map(|lock| lock.into_inner()),
);

let print_status = cfg!(debug_assertions) && hcx.debug_dep_tasks();
let print_status = cfg!(debug_assertions) && cx.debug_dep_tasks();

// Determine the color of the new DepNode.
if let Some(prev_index) = data.previous.node_to_index_opt(&key) {
Expand Down
2 changes: 2 additions & 0 deletions src/librustc_query_system/dep_graph/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,8 @@ pub trait DepContext: Copy {
/// Create a hashing context for hashing new results.
fn create_stable_hashing_context(&self) -> Self::StableHashingContext;

fn debug_dep_tasks(&self) -> bool;

/// Try to force a dep node to execute and see if it's green.
fn try_force_from_dep_node(&self, dep_node: &DepNode<Self::DepKind>) -> bool;

Expand Down
4 changes: 1 addition & 3 deletions src/librustc_query_system/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,4 @@ extern crate rustc_data_structures;
pub mod dep_graph;
pub mod query;

pub trait HashStableContext {
fn debug_dep_tasks(&self) -> bool;
}
pub trait HashStableContext {}

0 comments on commit 222d010

Please sign in to comment.