From a432f87d66bdfb8e9eef4f6d38d3d5efc2488c1f Mon Sep 17 00:00:00 2001 From: Aleksey Kladov Date: Fri, 24 Jul 2020 14:12:13 +0200 Subject: Cache macro expansion in semantics #5497 accidentally made syntax highlighting quadratic, due to repeated tokentreeizing of macros. --- crates/ra_hir/src/semantics.rs | 16 ++++++++++++++-- 1 file changed, 14 insertions(+), 2 deletions(-) (limited to 'crates/ra_hir/src') diff --git a/crates/ra_hir/src/semantics.rs b/crates/ra_hir/src/semantics.rs index 02d83e0d9..1436b1afe 100644 --- a/crates/ra_hir/src/semantics.rs +++ b/crates/ra_hir/src/semantics.rs @@ -89,6 +89,7 @@ pub struct Semantics<'db, DB> { pub struct SemanticsImpl<'db> { pub db: &'db dyn HirDatabase, s2d_cache: RefCell, + expansion_info_cache: RefCell>>, cache: RefCell>, } @@ -275,7 +276,12 @@ impl<'db, DB: HirDatabase> Semantics<'db, DB> { impl<'db> SemanticsImpl<'db> { fn new(db: &'db dyn HirDatabase) -> Self { - Self { db, s2d_cache: Default::default(), cache: Default::default() } + SemanticsImpl { + db, + s2d_cache: Default::default(), + cache: Default::default(), + expansion_info_cache: Default::default(), + } } fn parse(&self, file_id: FileId) -> ast::SourceFile { @@ -328,7 +334,13 @@ impl<'db> SemanticsImpl<'db> { return None; } let file_id = sa.expand(self.db, token.with_value(¯o_call))?; - let token = file_id.expansion_info(self.db.upcast())?.map_token_down(token.as_ref())?; + let token = self + .expansion_info_cache + .borrow_mut() + .entry(file_id) + .or_insert_with(|| file_id.expansion_info(self.db.upcast())) + .as_ref()? + .map_token_down(token.as_ref())?; self.cache(find_root(&token.value.parent()), token.file_id); -- cgit v1.2.3