[Gc] Re: [bdwgc] All interior pointers must be enabled for
finalized allocation. (#9)
ivmai at mail.ru
Mon Jan 16 05:08:28 PST 2012
There could be an alternative solution: GC_set_all_interior_pointers(TRUE) in GC_init_finalized_malloc (in this case, the comment should note that all-interior-mode should not be turned off after GC_init_finalized_malloc and there should be GC_ASSERT(GC_all_interior_pointers); in every malloc function in fnzl_mlc). This seems to me more logical as if we want some functionality why do we need to care about enabling others that are implementation-specific.
What do you think (prefer)?
Note also that DCL_LOCK_STATE goes before any statement.
13 01 2012, 22:44 Petter Urkedal <reply+i-2834854-83bd23260ff0c5e2eec2b637e8c7d3152ad12e13-460469 at reply.github.com>:
> * include/gc_disclaim.h: Add note about all-interior-pointers requirement.
> * fnlz_mlc.c (GC_init_finalized_malloc): Abort with a useful message if all
> interior pointers are not enabled.
> * tests/disclaim_bench.c, tests/disclaim_test.c: Enable all interior
> pointers for tests.
> You can merge this Pull Request by running:
> git pull https://github.com/paurkedal/bdwgc fnlz-requires-all-interior
> Or you can view, comment on it, or merge it online at:
> -- Commit Summary --
> * All interior pointers must be enabled for finalized allocation.
> -- File Changes --
> M fnlz_mlc.c (4)
> M include/gc_disclaim.h (3)
> M tests/disclaim_bench.c (1)
> M tests/disclaim_test.c (1)
> -- Patch Links --
> Reply to this email directly or view it on GitHub:
More information about the Gc