Skip to content

Commit 0d17de8

Browse files
authored
Rollup merge of #65176 - nnethercote:rm-query-macros, r=michaelwoerister
Remove query-related macros The query system has a few macros that only have one or two call sites, and I find they hurt readability. This PR removes them. r? @michaelwoerister
2 parents f2dc346 + 9267d9f commit 0d17de8

File tree

2 files changed

+18
-33
lines changed

2 files changed

+18
-33
lines changed

src/librustc/ty/query/plumbing.rs

+13-32
Original file line numberDiff line numberDiff line change
@@ -1183,37 +1183,6 @@ pub fn force_from_dep_node(tcx: TyCtxt<'_>, dep_node: &DepNode) -> bool {
11831183
return false
11841184
}
11851185

1186-
macro_rules! def_id {
1187-
() => {
1188-
if let Some(def_id) = dep_node.extract_def_id(tcx) {
1189-
def_id
1190-
} else {
1191-
// Return from the whole function.
1192-
return false
1193-
}
1194-
}
1195-
};
1196-
1197-
macro_rules! krate {
1198-
() => { (def_id!()).krate }
1199-
};
1200-
1201-
macro_rules! force_ex {
1202-
($tcx:expr, $query:ident, $key:expr) => {
1203-
{
1204-
$tcx.force_query::<crate::ty::query::queries::$query<'_>>(
1205-
$key,
1206-
DUMMY_SP,
1207-
*dep_node
1208-
);
1209-
}
1210-
}
1211-
};
1212-
1213-
macro_rules! force {
1214-
($query:ident, $key:expr) => { force_ex!(tcx, $query, $key) }
1215-
};
1216-
12171186
rustc_dep_node_force!([dep_node, tcx]
12181187
// These are inputs that are expected to be pre-allocated and that
12191188
// should therefore always be red or green already.
@@ -1232,7 +1201,19 @@ pub fn force_from_dep_node(tcx: TyCtxt<'_>, dep_node: &DepNode) -> bool {
12321201
bug!("force_from_dep_node: encountered {:?}", dep_node)
12331202
}
12341203

1235-
DepKind::Analysis => { force!(analysis, krate!()); }
1204+
DepKind::Analysis => {
1205+
let def_id = if let Some(def_id) = dep_node.extract_def_id(tcx) {
1206+
def_id
1207+
} else {
1208+
// Return from the whole function.
1209+
return false
1210+
};
1211+
tcx.force_query::<crate::ty::query::queries::analysis<'_>>(
1212+
def_id.krate,
1213+
DUMMY_SP,
1214+
*dep_node
1215+
);
1216+
}
12361217
);
12371218

12381219
true

src/librustc_macros/src/query.rs

+5-1
Original file line numberDiff line numberDiff line change
@@ -495,7 +495,11 @@ pub fn rustc_queries(input: TokenStream) -> TokenStream {
495495
dep_node_force_stream.extend(quote! {
496496
DepKind::#name => {
497497
if let Some(key) = RecoverKey::recover($tcx, $dep_node) {
498-
force_ex!($tcx, #name, key);
498+
$tcx.force_query::<crate::ty::query::queries::#name<'_>>(
499+
key,
500+
DUMMY_SP,
501+
*$dep_node
502+
);
499503
} else {
500504
return false;
501505
}

0 commit comments

Comments
 (0)