summaryrefslogtreecommitdiff
path: root/yjit
diff options
context:
space:
mode:
authorAlan Wu <XrXr@users.noreply.github.com>2022-11-29 15:41:25 -0500
committerMaxime Chevalier-Boisvert <maximechevalierb@gmail.com>2022-11-30 12:23:50 -0500
commitb30248f74a8f6ce37a78f07597c7c5452ff50abd (patch)
treed2ec76c1d6d161156879e86726c736d9581c6dd6 /yjit
parent03f1e6a2aa8aa1d5aef79a33a243372a457f0fa2 (diff)
YJIT: Deallocate when assumptions tables are empty
When we run global invalidation for TracePoints or code GC, we clear out all blocks in our assumptions table but we don't deallocate the backing buffers. Let's reclaim some memory during these rare events.
Notes
Notes: Merged: https://github.com/ruby/ruby/pull/6833
Diffstat (limited to 'yjit')
-rw-r--r--yjit/src/invariants.rs28
1 files changed, 28 insertions, 0 deletions
diff --git a/yjit/src/invariants.rs b/yjit/src/invariants.rs
index 1542b40db8..5a44ce1bfc 100644
--- a/yjit/src/invariants.rs
+++ b/yjit/src/invariants.rs
@@ -357,8 +357,14 @@ pub fn block_assumptions_free(blockref: &BlockRef) {
// Remove tracking for cme validity
if let Some(blockset) = invariants.cme_validity.get_mut(dep) {
blockset.remove(blockref);
+ if blockset.is_empty() {
+ invariants.cme_validity.remove(dep);
+ }
}
}
+ if invariants.cme_validity.is_empty() {
+ invariants.cme_validity.shrink_to_fit();
+ }
}
// Remove tracking for basic operators that the given block assumes have
@@ -369,20 +375,42 @@ pub fn block_assumptions_free(blockref: &BlockRef) {
for key in &bops {
if let Some(blocks) = invariants.basic_operator_blocks.get_mut(key) {
blocks.remove(&blockref);
+ if blocks.is_empty() {
+ invariants.basic_operator_blocks.remove(key);
+ }
}
}
}
+ if invariants.block_basic_operators.is_empty() {
+ invariants.block_basic_operators.shrink_to_fit();
+ }
+ if invariants.basic_operator_blocks.is_empty() {
+ invariants.basic_operator_blocks.shrink_to_fit();
+ }
+ // Remove tracking for blocks assuming single ractor mode
invariants.single_ractor.remove(&blockref);
+ if invariants.single_ractor.is_empty() {
+ invariants.single_ractor.shrink_to_fit();
+ }
// Remove tracking for constant state for a given ID.
if let Some(ids) = invariants.block_constant_states.remove(&blockref) {
for id in ids {
if let Some(blocks) = invariants.constant_state_blocks.get_mut(&id) {
blocks.remove(&blockref);
+ if blocks.is_empty() {
+ invariants.constant_state_blocks.remove(&id);
+ }
}
}
}
+ if invariants.block_constant_states.is_empty() {
+ invariants.block_constant_states.shrink_to_fit();
+ }
+ if invariants.constant_state_blocks.is_empty() {
+ invariants.constant_state_blocks.shrink_to_fit();
+ }
}
/// Callback from the opt_setinlinecache instruction in the interpreter.