roland/xo-umbrella2
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

xo-gc: retire Collector.forward_inplace + mock x-gc utest

Browse source 
W/ MockCollector supporting AGCObjectVisitor facet
This commit is contained in:
Roland Conybeare 2026-04-07 21:16:55 -04:00
commit 1387673f15
14 changed files with 342 additions and 42 deletions
Download patch file Download diff file Expand all files Collapse all files

31
src/gc/DX1Collector.cpp
View file

@ -584,32 +584,13 @@ namespace xo {
}
}
void
DX1Collector::forward_inplace(AGCObject * lhs_iface,
void ** lhs_data)
{
// TODO: streamline once GCObject refactored so that
// forward_children takes GCObjectVisitor instead of Collector
// argument.
Generation upto = runstate_.gc_upto();
if (runstate_.is_running()) {
// called during collection phase
gco_store_.forward_inplace_aux(this->ref<AGCObjectVisitor>(), lhs_iface, lhs_data, upto);
} else if (runstate_.is_verify()) {
// called during verify_ok
this->_verify_aux(lhs_iface, *lhs_data);
} else {
// should be unreachable
assert(false);
}
}
void
DX1Collector::visit_child(AGCObject * lhs_iface,
void ** lhs_data)
{
// MAYBE: adapter distinct from DX1Collector that supports GCObjectVisitor facet,
// calls DX1Collector::_verify_aux()
if (runstate_.is_running()) {
Generation upto = runstate_.gc_upto();
@ -661,17 +642,17 @@ namespace xo {
auto
DX1Collector::super_alloc(typeseq t, size_type z) noexcept -> value_type {
return with_facet<AAllocator>::mkobj(new_space()).super_alloc(t, z);
return with_facet<AAllocator>::mkobj(this->new_space()).super_alloc(t, z);
}
auto
DX1Collector::sub_alloc(size_type z, bool complete) noexcept -> value_type {
return with_facet<AAllocator>::mkobj(new_space()).sub_alloc(z, complete);
return with_facet<AAllocator>::mkobj(this->new_space()).sub_alloc(z, complete);
}
auto
DX1Collector::alloc_copy(value_type src) noexcept -> value_type {
return with_facet<AAllocator>::mkobj(new_space()).alloc_copy(src);
return with_facet<AAllocator>::mkobj(this->new_space()).alloc_copy(src);
}
bool