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

Pin shape->edges

Browse Source
This commit is contained in:
Jean Boussier 2025-06-02 14:45:02 +02:00
parent e9fd44dd72
commit db2cfebff1
Notes: git 2025-06-02 15:50:07 +00:00 
Merged: https://github.com/ruby/ruby/pull/13441
1 changed files with 8 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
shape.c
View File

@ -324,7 +324,14 @@ shape_tree_mark(void *data)
rb_shape_t *end = RSHAPE(GET_SHAPE_TREE()->next_shape_id);
while (cursor < end) {
if (cursor->edges && !SINGLE_CHILD_P(cursor->edges)) {
rb_gc_mark_movable(cursor->edges);
// FIXME: GC compaction may call `rb_shape_traverse_from_new_root`
// to migrate objects from one object slot to another.
// Because of this if we don't pin `cursor->edges` it might be turned
// into a T_MOVED during GC.
// We'd need to eliminate `SHAPE_T_OBJECT` so that GC never need to lookup
// shapes this way.
// rb_gc_mark_movable(cursor->edges);
rb_gc_mark(cursor->edges);
}
cursor++;
}