Eliminate second GC pass for eliminating T_MOVED

`T_MOVED` is a linked list, so we can just iterate through the `T_MOVED`
objects, clearing them out and adding them to respective free lists.
This commit is contained in:
Aaron Patterson 2019-10-07 10:14:13 -07:00
Родитель d8b6f58502
Коммит 0a2f04e156
Не найден ключ, соответствующий данной подписи
Идентификатор ключа GPG: 953170BCB4FFAFC6
1 изменённых файлов: 13 добавлений и 10 удалений

23
gc.c
Просмотреть файл

@ -8401,20 +8401,23 @@ gc_compact_after_gc(rb_objspace_t *objspace, int use_toward_empty, int use_doubl
gc_verify_internal_consistency(objspace);
}
#if __has_feature(address_sanitizer)
while (moved_list) {
VALUE current = moved_list;
moved_list = RANY(moved_list)->as.moved.next;
asan_poison_object(current);
VALUE next_moved;
struct heap_page *page;
page = GET_HEAP_PAGE(moved_list);
next_moved = RMOVED(moved_list)->next;
RMOVED(moved_list)->flags = 0;
RMOVED(moved_list)->destination = 0;
RMOVED(moved_list)->next = 0;
page->free_slots++;
heap_page_add_freeobj(objspace, page, moved_list);
objspace->profile.total_freed_objects++;
moved_list = next_moved;
}
#else
(void)moved_list;
#endif
mjit_gc_exit_hook(); // unlock MJIT here, because `rb_gc()` calls `mjit_gc_start_hook()` again.
/* GC after compaction to eliminate T_MOVED */
garbage_collect(objspace, GPR_DEFAULT_REASON);
}
/*