Merge pull request #90 from ruby/obj-free-needs-cleanup

peterzhu2118 · web-flow · commit b8db698dd2fa · 2026-06-04T11:54:10.000+09:00
Use rb_gc_obj_needs_cleanup_p
diff --git a/gc/mmtk/mmtk.c b/gc/mmtk/mmtk.c
@@ -345,7 +345,9 @@ rb_mmtk_call_obj_free(MMTk_ObjectReference object)
         pthread_mutex_unlock(&objspace->event_hook_mutex);
     }
 
-    rb_gc_obj_free(objspace, obj);
+    if (RB_UNLIKELY(rb_gc_obj_needs_cleanup_p(obj))) {
+        rb_gc_obj_free(objspace, obj);
+    }
 
 #ifdef MMTK_DEBUG
     memset((void *)obj, 0, rb_gc_impl_obj_slot_size(obj));

Original file line number	Diff line number	Diff line change
`@@ -345,7 +345,9 @@ rb_mmtk_call_obj_free(MMTk_ObjectReference object)`
`345`	`345`	`pthread_mutex_unlock(&objspace->event_hook_mutex);`
`346`	`346`	`}`
`347`	`347`
`348`		`- rb_gc_obj_free(objspace, obj);`
	`348`	`+ if (RB_UNLIKELY(rb_gc_obj_needs_cleanup_p(obj))) {`
	`349`	`+ rb_gc_obj_free(objspace, obj);`
	`350`	`+ }`
`349`	`351`
`350`	`352`	`#ifdef MMTK_DEBUG`
`351`	`353`	`memset((void *)obj, 0, rb_gc_impl_obj_slot_size(obj));`