Skip to content

Commit 855282f

Browse files
VeykrilVeykril
committed
internal: Don't unnecessarily clone ModPaths in early name res
1 parent 72dfbe9 commit 855282f

File tree

12 files changed

+29
-30
lines changed

12 files changed

+29
-30
lines changed

‎crates/hir-def/src/body/lower.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -965,7 +965,7 @@ impl ExprCollector<'_> {
965965
.resolve_path(
966966
self.db,
967967
module,
968-
&path,
968+
path,
969969
crate::item_scope::BuiltinShadowMode::Other,
970970
Some(MacroSubNs::Bang),
971971
)

‎crates/hir-def/src/data.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -719,12 +719,12 @@ impl<'a> AssocItemCollector<'a> {
719719
let MacroCall { ast_id, expand_to, ctxt, ref path } = item_tree[call];
720720
let module = self.expander.module.local_id;
721721

722-
let resolver = |path| {
722+
let resolver = |path: &_| {
723723
self.def_map
724724
.resolve_path(
725725
self.db,
726726
module,
727-
&path,
727+
path,
728728
crate::item_scope::BuiltinShadowMode::Other,
729729
Some(MacroSubNs::Bang),
730730
)

‎crates/hir-def/src/expander.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@ impl Expander {
5656
&mut self,
5757
db: &dyn DefDatabase,
5858
macro_call: ast::MacroCall,
59-
resolver: impl Fn(ModPath) -> Option<MacroId>,
59+
resolver: impl Fn(&ModPath) -> Option<MacroId>,
6060
) -> Result<ExpandResult<Option<(Mark, Parse<T>)>>, UnresolvedMacro> {
6161
// FIXME: within_limit should support this, instead of us having to extract the error
6262
let mut unresolved_macro_err = None;

‎crates/hir-def/src/generics.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -403,12 +403,12 @@ impl GenericParamsCollector {
403403
let (def_map, expander) = &mut **exp;
404404

405405
let module = expander.module.local_id;
406-
let resolver = |path| {
406+
let resolver = |path: &_| {
407407
def_map
408408
.resolve_path(
409409
db,
410410
module,
411-
&path,
411+
path,
412412
crate::item_scope::BuiltinShadowMode::Other,
413413
Some(MacroSubNs::Bang),
414414
)

‎crates/hir-def/src/lib.rs

Lines changed: 7 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1370,7 +1370,7 @@ pub trait AsMacroCall {
13701370
&self,
13711371
db: &dyn ExpandDatabase,
13721372
krate: CrateId,
1373-
resolver: impl Fn(path::ModPath) -> Option<MacroDefId> + Copy,
1373+
resolver: impl Fn(&path::ModPath) -> Option<MacroDefId> + Copy,
13741374
) -> Option<MacroCallId> {
13751375
self.as_call_id_with_errors(db, krate, resolver).ok()?.value
13761376
}
@@ -1379,7 +1379,7 @@ pub trait AsMacroCall {
13791379
&self,
13801380
db: &dyn ExpandDatabase,
13811381
krate: CrateId,
1382-
resolver: impl Fn(path::ModPath) -> Option<MacroDefId> + Copy,
1382+
resolver: impl Fn(&path::ModPath) -> Option<MacroDefId> + Copy,
13831383
) -> Result<ExpandResult<Option<MacroCallId>>, UnresolvedMacro>;
13841384
}
13851385

@@ -1388,7 +1388,7 @@ impl AsMacroCall for InFile<&ast::MacroCall> {
13881388
&self,
13891389
db: &dyn ExpandDatabase,
13901390
krate: CrateId,
1391-
resolver: impl Fn(path::ModPath) -> Option<MacroDefId> + Copy,
1391+
resolver: impl Fn(&path::ModPath) -> Option<MacroDefId> + Copy,
13921392
) -> Result<ExpandResult<Option<MacroCallId>>, UnresolvedMacro> {
13931393
let expands_to = hir_expand::ExpandTo::from_call_site(self.value);
13941394
let ast_id = AstId::new(self.file_id, db.ast_id_map(self.file_id).ast_id(self.value));
@@ -1437,7 +1437,7 @@ fn macro_call_as_call_id(
14371437
call_site: SyntaxContextId,
14381438
expand_to: ExpandTo,
14391439
krate: CrateId,
1440-
resolver: impl Fn(path::ModPath) -> Option<MacroDefId> + Copy,
1440+
resolver: impl Fn(&path::ModPath) -> Option<MacroDefId> + Copy,
14411441
) -> Result<Option<MacroCallId>, UnresolvedMacro> {
14421442
macro_call_as_call_id_with_eager(db, call, call_site, expand_to, krate, resolver, resolver)
14431443
.map(|res| res.value)
@@ -1449,11 +1449,10 @@ fn macro_call_as_call_id_with_eager(
14491449
call_site: SyntaxContextId,
14501450
expand_to: ExpandTo,
14511451
krate: CrateId,
1452-
resolver: impl FnOnce(path::ModPath) -> Option<MacroDefId>,
1453-
eager_resolver: impl Fn(path::ModPath) -> Option<MacroDefId>,
1452+
resolver: impl FnOnce(&path::ModPath) -> Option<MacroDefId>,
1453+
eager_resolver: impl Fn(&path::ModPath) -> Option<MacroDefId>,
14541454
) -> Result<ExpandResult<Option<MacroCallId>>, UnresolvedMacro> {
1455-
let def =
1456-
resolver(call.path.clone()).ok_or_else(|| UnresolvedMacro { path: call.path.clone() })?;
1455+
let def = resolver(&call.path).ok_or_else(|| UnresolvedMacro { path: call.path.clone() })?;
14571456

14581457
let res = match def.kind {
14591458
MacroDefKind::BuiltInEager(..) => expand_eager_macro_input(

‎crates/hir-def/src/macro_expansion_tests/mod.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -96,7 +96,7 @@ pub fn identity_when_valid(_attr: TokenStream, item: TokenStream) -> TokenStream
9696
let res = macro_call
9797
.as_call_id_with_errors(&db, krate, |path| {
9898
resolver
99-
.resolve_path_as_macro(&db, &path, Some(MacroSubNs::Bang))
99+
.resolve_path_as_macro(&db, path, Some(MacroSubNs::Bang))
100100
.map(|(it, _)| db.macro_def(it))
101101
})
102102
.unwrap();

‎crates/hir-def/src/nameres/attr_resolution.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -137,10 +137,10 @@ pub(super) fn derive_macro_as_call_id(
137137
derive_pos: u32,
138138
call_site: SyntaxContextId,
139139
krate: CrateId,
140-
resolver: impl Fn(path::ModPath) -> Option<(MacroId, MacroDefId)>,
140+
resolver: impl Fn(&path::ModPath) -> Option<(MacroId, MacroDefId)>,
141141
derive_macro_id: MacroCallId,
142142
) -> Result<(MacroId, MacroDefId, MacroCallId), UnresolvedMacro> {
143-
let (macro_id, def_id) = resolver(item_attr.path.clone())
143+
let (macro_id, def_id) = resolver(&item_attr.path)
144144
.filter(|(_, def_id)| def_id.is_derive())
145145
.ok_or_else(|| UnresolvedMacro { path: item_attr.path.clone() })?;
146146
let call_id = def_id.make_call(

‎crates/hir-def/src/nameres/collector.rs

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1137,18 +1137,18 @@ impl DefCollector<'_> {
11371137
MacroSubNs::Attr
11381138
}
11391139
};
1140-
let resolver = |path| {
1140+
let resolver = |path: &_| {
11411141
let resolved_res = self.def_map.resolve_path_fp_with_macro(
11421142
self.db,
11431143
ResolveMode::Other,
11441144
directive.module_id,
1145-
&path,
1145+
path,
11461146
BuiltinShadowMode::Module,
11471147
Some(subns),
11481148
);
11491149
resolved_res.resolved_def.take_macros().map(|it| (it, self.db.macro_def(it)))
11501150
};
1151-
let resolver_def_id = |path| resolver(path).map(|(_, it)| it);
1151+
let resolver_def_id = |path: &_| resolver(path).map(|(_, it)| it);
11521152

11531153
match &directive.kind {
11541154
MacroDirectiveKind::FnLike { ast_id, expand_to, ctxt: call_site } => {
@@ -1251,7 +1251,7 @@ impl DefCollector<'_> {
12511251
}
12521252
}
12531253

1254-
let def = match resolver_def_id(path.clone()) {
1254+
let def = match resolver_def_id(path) {
12551255
Some(def) if def.is_attribute() => def,
12561256
_ => return Resolved::No,
12571257
};
@@ -1439,7 +1439,7 @@ impl DefCollector<'_> {
14391439
self.db,
14401440
ResolveMode::Other,
14411441
directive.module_id,
1442-
&path,
1442+
path,
14431443
BuiltinShadowMode::Module,
14441444
Some(MacroSubNs::Bang),
14451445
);
@@ -2339,7 +2339,7 @@ impl ModCollector<'_, '_> {
23392339
db,
23402340
ResolveMode::Other,
23412341
self.module_id,
2342-
&path,
2342+
path,
23432343
BuiltinShadowMode::Module,
23442344
Some(MacroSubNs::Bang),
23452345
);

‎crates/hir-expand/src/eager.rs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ pub fn expand_eager_macro_input(
3939
ast_id: AstId<ast::MacroCall>,
4040
def: MacroDefId,
4141
call_site: SyntaxContextId,
42-
resolver: &dyn Fn(ModPath) -> Option<MacroDefId>,
42+
resolver: &dyn Fn(&ModPath) -> Option<MacroDefId>,
4343
) -> ExpandResult<Option<MacroCallId>> {
4444
let expand_to = ExpandTo::from_call_site(macro_call);
4545

@@ -138,7 +138,7 @@ fn eager_macro_recur(
138138
curr: InFile<SyntaxNode>,
139139
krate: CrateId,
140140
call_site: SyntaxContextId,
141-
macro_resolver: &dyn Fn(ModPath) -> Option<MacroDefId>,
141+
macro_resolver: &dyn Fn(&ModPath) -> Option<MacroDefId>,
142142
) -> ExpandResult<Option<(SyntaxNode, TextSize)>> {
143143
let original = curr.value.clone_for_update();
144144

@@ -172,7 +172,7 @@ fn eager_macro_recur(
172172
let def = match call.path().and_then(|path| {
173173
ModPath::from_src(db, path, &mut |range| span_map.span_at(range.start()).ctx)
174174
}) {
175-
Some(path) => match macro_resolver(path.clone()) {
175+
Some(path) => match macro_resolver(&path) {
176176
Some(def) => def,
177177
None => {
178178
error =

‎crates/hir-ty/src/lower.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -416,9 +416,9 @@ impl<'a> TyLoweringContext<'a> {
416416
};
417417
let ty = {
418418
let macro_call = macro_call.to_node(self.db.upcast());
419-
let resolver = |path| {
419+
let resolver = |path: &_| {
420420
self.resolver
421-
.resolve_path_as_macro(self.db.upcast(), &path, Some(MacroSubNs::Bang))
421+
.resolve_path_as_macro(self.db.upcast(), path, Some(MacroSubNs::Bang))
422422
.map(|(it, _)| it)
423423
};
424424
match expander.enter_expand::<ast::Type>(self.db.upcast(), macro_call, resolver)

‎crates/hir/src/semantics.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -430,7 +430,7 @@ impl<'db> SemanticsImpl<'db> {
430430
let macro_call = InFile::new(file_id, actual_macro_call);
431431
let krate = resolver.krate();
432432
let macro_call_id = macro_call.as_call_id(self.db.upcast(), krate, |path| {
433-
resolver.resolve_path_as_macro_def(self.db.upcast(), &path, Some(MacroSubNs::Bang))
433+
resolver.resolve_path_as_macro_def(self.db.upcast(), path, Some(MacroSubNs::Bang))
434434
})?;
435435
hir_expand::db::expand_speculative(
436436
self.db.upcast(),

‎crates/hir/src/source_analyzer.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -826,7 +826,7 @@ impl SourceAnalyzer {
826826
// FIXME: This causes us to parse, generally this is the wrong approach for resolving a
827827
// macro call to a macro call id!
828828
let macro_call_id = macro_call.as_call_id(db.upcast(), krate, |path| {
829-
self.resolver.resolve_path_as_macro_def(db.upcast(), &path, Some(MacroSubNs::Bang))
829+
self.resolver.resolve_path_as_macro_def(db.upcast(), path, Some(MacroSubNs::Bang))
830830
})?;
831831
// why the 64?
832832
Some(macro_call_id.as_macro_file()).filter(|it| it.expansion_level(db.upcast()) < 64)

0 commit comments

Comments
 (0)