Skip to content

Commit 820a028

Browse files
committed
MMTk: Rename GIVTBL to gen fields tbl
The upstream renamed `generic_iv_tbl_` to `generic_fields_tbl_`, and the struct `gen_ivtbl` to `gen_fields_tbl`. We rename accordingly.
1 parent 5b3158d commit 820a028

File tree

3 files changed

+29
-28
lines changed

3 files changed

+29
-28
lines changed

internal/mmtk.h

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -80,16 +80,17 @@ typedef struct MMTk_RubyUpcalls {
8080
void (*scan_object_ruby_style)(MMTk_ObjectReference object);
8181
void (*call_gc_mark_children)(MMTk_ObjectReference object);
8282
void (*call_obj_free)(MMTk_ObjectReference object);
83-
void (*cleanup_generic_iv_tbl)(void);
84-
void *(*get_original_givtbl)(MMTk_ObjectReference object);
85-
void (*move_givtbl)(MMTk_ObjectReference old_objref, MMTk_ObjectReference new_objref);
83+
void (*cleanup_generic_fields_tbl)(void);
84+
void *(*get_original_gen_fields_tbl)(MMTk_ObjectReference object);
85+
void (*reinsert_generic_fields_tbl_entry)(MMTk_ObjectReference old_objref,
86+
MMTk_ObjectReference new_objref);
8687
size_t (*vm_live_bytes)(void);
8788
void (*update_frozen_strings_table)(void);
8889
void (*update_finalizer_and_obj_id_tables)(void);
8990
void (*update_global_symbols_table)(void);
9091
void (*update_overloaded_cme_table)(void);
9192
void (*update_ci_table)(void);
92-
struct st_table *(*get_generic_iv_tbl)(void);
93+
struct st_table *(*get_generic_fields_tbl)(void);
9394
size_t (*get_num_fstrings)(void);
9495
struct st_table *(*get_finalizer_table)(void);
9596
struct st_table *(*get_id2ref_table)(void);
@@ -242,7 +243,7 @@ void mmtk_register_ppp(MMTk_ObjectReference object);
242243

243244
void mmtk_register_ppps(const MMTk_ObjectReference *objects, size_t len);
244245

245-
void *mmtk_get_givtbl_during_gc(MMTk_ObjectReference object);
246+
void *mmtk_get_gen_fields_tbl_during_gc(MMTk_ObjectReference object);
246247

247248
size_t mmtk_get_vo_bit_log_region_size(void);
248249

mmtk_support.c

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1742,7 +1742,7 @@ rb_mmtk_scan_roots_in_mutator_thread(MMTk_VMMutatorThread vm_mutator, MMTk_VMWor
17421742
}
17431743

17441744
static void*
1745-
rb_mmtk_get_original_givtbl(MMTk_ObjectReference object) {
1745+
rb_mmtk_get_original_gen_fields_tbl(MMTk_ObjectReference object) {
17461746
VALUE obj = (VALUE)object;
17471747

17481748
RUBY_ASSERT(FL_TEST(obj, FL_EXIVAR));
@@ -1752,18 +1752,18 @@ rb_mmtk_get_original_givtbl(MMTk_ObjectReference object) {
17521752
struct st_table *rb_generic_fields_tbl_get(void); // Defined in variable.c
17531753

17541754
st_table*
1755-
rb_mmtk_get_generic_iv_tbl(void) {
1755+
rb_mmtk_get_generic_fields_tbl(void) {
17561756
return rb_generic_fields_tbl_get();
17571757
}
17581758

1759-
void rb_mmtk_mv_generic_ivar(VALUE src, VALUE dst); // Defined in variable.c
1759+
void rb_mmtk_reinsert_generic_fields_tbl_entry(VALUE src, VALUE dst); // Defined in variable.c
17601760

17611761
static void
1762-
rb_mmtk_move_givtbl(MMTk_ObjectReference old_objref, MMTk_ObjectReference new_objref) {
1763-
rb_mmtk_mv_generic_ivar((VALUE)old_objref, (VALUE)new_objref);
1762+
rb_mmtk_reinsert_generic_fields_tbl_entry_wrapper(MMTk_ObjectReference old_objref, MMTk_ObjectReference new_objref) {
1763+
rb_mmtk_reinsert_generic_fields_tbl_entry((VALUE)old_objref, (VALUE)new_objref);
17641764
}
17651765

1766-
void rb_mmtk_cleanup_generic_iv_tbl(void); // Defined in variable.c
1766+
void rb_mmtk_cleanup_generic_fields_tbl(void); // Defined in variable.c
17671767
void rb_mmtk_update_frozen_strings_table(void); // Defined in default.c
17681768
void rb_mmtk_update_finalizer_and_obj_id_tables(void); // Defined in default.c
17691769
void rb_mmtk_update_global_symbols_table(void); // Defined in gc.c
@@ -1799,16 +1799,16 @@ MMTk_RubyUpcalls ruby_upcalls = {
17991799
rb_mmtk_scan_object_ruby_style,
18001800
rb_mmtk_call_gc_mark_children,
18011801
rb_mmtk_call_obj_free,
1802-
rb_mmtk_cleanup_generic_iv_tbl,
1803-
rb_mmtk_get_original_givtbl,
1804-
rb_mmtk_move_givtbl,
1802+
rb_mmtk_cleanup_generic_fields_tbl,
1803+
rb_mmtk_get_original_gen_fields_tbl,
1804+
rb_mmtk_reinsert_generic_fields_tbl_entry_wrapper,
18051805
rb_mmtk_vm_live_bytes,
18061806
rb_mmtk_update_frozen_strings_table,
18071807
rb_mmtk_update_finalizer_and_obj_id_tables,
18081808
rb_mmtk_update_global_symbols_table,
18091809
rb_mmtk_update_overloaded_cme_table,
18101810
rb_mmtk_update_ci_table,
1811-
rb_mmtk_get_generic_iv_tbl,
1811+
rb_mmtk_get_generic_fields_tbl,
18121812
rb_mmtk_debug_get_num_fstrings,
18131813
rb_mmtk_get_finalizer_table,
18141814
rb_mmtk_get_id2ref_table,

variable.c

Lines changed: 13 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -1303,7 +1303,7 @@ rb_mark_generic_ivar(VALUE obj)
13031303
void
13041304
rb_mmtk_update_generic_ivar(VALUE obj)
13051305
{
1306-
struct gen_fields_tbl *fields_tbl = (struct gen_fields_tbl *)mmtk_get_givtbl_during_gc((MMTk_ObjectReference)obj);
1306+
struct gen_fields_tbl *fields_tbl = (struct gen_fields_tbl *)mmtk_get_gen_fields_tbl_during_gc((MMTk_ObjectReference)obj);
13071307
if (fields_tbl != NULL) {
13081308
if (rb_shape_obj_too_complex_p(obj)) {
13091309
rb_gc_update_tbl_refs(fields_tbl->as.complex.table);
@@ -1317,22 +1317,22 @@ rb_mmtk_update_generic_ivar(VALUE obj)
13171317
}
13181318

13191319
void
1320-
rb_mmtk_mv_generic_ivar(VALUE rsrc, VALUE dst)
1320+
rb_mmtk_reinsert_generic_fields_tbl_entry(VALUE rsrc, VALUE dst)
13211321
{
13221322
st_data_t key = (st_data_t)rsrc;
1323-
st_data_t ivtbl;
1323+
st_data_t fields_tbl;
13241324

1325-
if (st_delete(generic_fields_tbl_no_ractor_check(rsrc), &key, &ivtbl))
1326-
st_insert(generic_fields_tbl_no_ractor_check(dst), (st_data_t)dst, ivtbl);
1325+
if (st_delete(generic_fields_tbl_no_ractor_check(rsrc), &key, &fields_tbl))
1326+
st_insert(generic_fields_tbl_no_ractor_check(dst), (st_data_t)dst, fields_tbl);
13271327
}
13281328

13291329
static int
1330-
rb_mmtk_cleanup_generic_iv_tbl_check(st_data_t key, st_data_t value, st_data_t argp, int error)
1330+
rb_mmtk_cleanup_generic_fields_tbl_check(st_data_t key, st_data_t value, st_data_t argp, int error)
13311331
{
13321332
MMTk_ObjectReference key_objref = (MMTk_ObjectReference)key;
1333-
// Delete gen_ivtbl for dead objects.
1333+
// Delete gen_fields_tbl for dead objects.
13341334
if (!mmtk_is_live_object(key_objref)) {
1335-
struct gen_ivtbl *tbl = (struct gen_ivtbl*)value;
1335+
struct gen_fields_tbl *tbl = (struct gen_fields_tbl*)value;
13361336
RUBY_ASSERT(tbl != NULL);
13371337
xfree(tbl);
13381338

@@ -1349,15 +1349,15 @@ rb_mmtk_cleanup_generic_iv_tbl_check(st_data_t key, st_data_t value, st_data_t a
13491349
}
13501350

13511351
/**
1352-
* Remove entries in generic_iv_tbl_ where the key is dead.
1353-
* Only used when using MMTk. In vanilla Ruby, the gen_ivtbl of an object is
1354-
* removed from generic_iv_tbl_ in obj_free when the object dies.
1352+
* Remove entries in generic_fields_tbl_ where the key is dead.
1353+
* Only used when using MMTk. In vanilla Ruby, the gen_fields_tbl of an object is
1354+
* removed from generic_fields_tbl_ in obj_free when the object dies.
13551355
*/
13561356
void
1357-
rb_mmtk_cleanup_generic_iv_tbl(void)
1357+
rb_mmtk_cleanup_generic_fields_tbl(void)
13581358
{
13591359
st_foreach_with_replace(generic_fields_tbl_,
1360-
rb_mmtk_cleanup_generic_iv_tbl_check,
1360+
rb_mmtk_cleanup_generic_fields_tbl_check,
13611361
NULL,
13621362
0);
13631363
}

0 commit comments

Comments
 (0)