From c04d4846f2a7a49e592f4e98f3e604851d600d4e Mon Sep 17 00:00:00 2001 From: Eduard Burtescu Date: Sun, 9 Mar 2014 00:18:58 +0200 Subject: [PATCH] Discard MTWT & interner tables from TLD after they stop being useful. --- src/librustc/driver/driver.rs | 7 +++++++ src/libsyntax/ext/mtwt.rs | 9 +++++++++ src/libsyntax/util/interner.rs | 10 ++++++++++ 3 files changed, 26 insertions(+) diff --git a/src/librustc/driver/driver.rs b/src/librustc/driver/driver.rs index 94965e7e0dc3f..1dc9e6b654581 100644 --- a/src/librustc/driver/driver.rs +++ b/src/librustc/driver/driver.rs @@ -305,6 +305,9 @@ pub fn phase_3_run_analysis_passes(sess: Session, time(time_passes, "resolution", (), |_| middle::resolve::resolve_crate(&sess, lang_items, krate)); + // Discard MTWT tables that aren't required past resolution. + syntax::ext::mtwt::clear_tables(); + let named_region_map = time(time_passes, "lifetime resolution", (), |_| middle::resolve_lifetime::krate(&sess, krate)); @@ -585,6 +588,10 @@ pub fn compile_input(sess: Session, cfg: ast::CrateConfig, input: &Input, if stop_after_phase_3(&analysis.ty_cx.sess) { return; } let (tcx, trans) = phase_4_translate_to_llvm(expanded_crate, analysis, &outputs); + + // Discard interned strings as they are no longer required. + token::get_ident_interner().clear(); + (outputs, trans, tcx.sess) }; phase_5_run_llvm_passes(&sess, &trans, &outputs); diff --git a/src/libsyntax/ext/mtwt.rs b/src/libsyntax/ext/mtwt.rs index 909f3eaf361e0..1a5587433ac59 100644 --- a/src/libsyntax/ext/mtwt.rs +++ b/src/libsyntax/ext/mtwt.rs @@ -126,6 +126,15 @@ pub fn display_sctable(table: &SCTable) { } } +/// Clear the tables from TLD to reclaim memory. +pub fn clear_tables() { + with_sctable(|table| { + *table.table.borrow_mut().get() = Vec::new(); + *table.mark_memo.borrow_mut().get() = HashMap::new(); + *table.rename_memo.borrow_mut().get() = HashMap::new(); + }); + with_resolve_table_mut(|table| *table = HashMap::new()); +} // Add a value to the end of a vec, return its index fn idx_push(vec: &mut Vec , val: T) -> u32 { diff --git a/src/libsyntax/util/interner.rs b/src/libsyntax/util/interner.rs index 969c7cec87cce..c1ed96fe4de48 100644 --- a/src/libsyntax/util/interner.rs +++ b/src/libsyntax/util/interner.rs @@ -84,6 +84,11 @@ impl Interner { None => None, } } + + pub fn clear(&self) { + *self.map.borrow_mut().get() = HashMap::new(); + *self.vect.borrow_mut().get() = Vec::new(); + } } #[deriving(Clone, Eq, Hash, Ord)] @@ -222,6 +227,11 @@ impl StrInterner { None => None, } } + + pub fn clear(&self) { + *self.map.borrow_mut().get() = HashMap::new(); + *self.vect.borrow_mut().get() = Vec::new(); + } } #[cfg(test)]