1berry/ruby
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Update ISeq references in stale_units as well

Browse Source 
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@67628 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
This commit is contained in:
k0kubun 2019-04-20 03:29:20 +00:00
parent b8b503d657
commit 27f75cf3dc
1 changed files with 11 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
mjit.c
View File

@ -116,10 +116,21 @@ mjit_update_references(const rb_iseq_t *iseq)
{ {
if (!mjit_enabled) if (!mjit_enabled)
return; return;
CRITICAL_SECTION_START(4, "mjit_free_iseq"); CRITICAL_SECTION_START(4, "mjit_free_iseq");
if (iseq->body->jit_unit) { if (iseq->body->jit_unit) {
iseq->body->jit_unit->iseq = (rb_iseq_t *)rb_gc_new_location((VALUE)iseq->body->jit_unit->iseq); iseq->body->jit_unit->iseq = (rb_iseq_t *)rb_gc_new_location((VALUE)iseq->body->jit_unit->iseq);
} }
// Units in stale_units (list of over-speculated and invalidated code) are not referenced from
// `iseq->body->jit_unit` anymore (because new one replaces that). So we need to check them too.
// TODO: we should be able to reduce the number of units checked here.
struct rb_mjit_unit *unit = NULL;
list_for_each(&stale_units.head, unit, unode) {
if (unit->iseq == iseq) {
unit->iseq = (rb_iseq_t *)rb_gc_new_location((VALUE)unit->iseq);
}
}
CRITICAL_SECTION_FINISH(4, "mjit_free_iseq"); CRITICAL_SECTION_FINISH(4, "mjit_free_iseq");
} }