From b43a56f9b50d4bcffdefb6179d2ca3d1a8fb1dbf Mon Sep 17 00:00:00 2001 From: Gareth Rees Date: Fri, 4 Sep 2015 15:24:14 +0100 Subject: [PATCH] Move arenadefaultzoneset to be adjacent to its todo. Add comments for ArenaDefaultZONESET and LocusPrefDEFAULT. Copied from Perforce Change: 188237 ServerID: perforce.ravenbrook.com --- mps/code/config.h | 15 ++++++++++----- 1 file changed, 10 insertions(+), 5 deletions(-) diff --git a/mps/code/config.h b/mps/code/config.h index 589fbc20c6b..d545b3ec0b4 100644 --- a/mps/code/config.h +++ b/mps/code/config.h @@ -406,8 +406,6 @@ #define ARENA_DEFAULT_ZONED TRUE -#define ArenaDefaultZONESET (ZoneSetUNIV << (MPS_WORD_WIDTH / 2)) - /* ARENA_MINIMUM_COLLECTABLE_SIZE is the minimum size (in bytes) of * collectable memory that might be considered worthwhile to run a * full garbage collection. */ @@ -431,9 +429,16 @@ #define ARENA_MAX_COLLECT_FRACTION (0.1) -/* TODO: This is left over from before the branch/2014-01-29/mps-chain-zones - and 2014-01-17/cbs-tract-alloc reformed allocation, and may now be doing - more harm than good. Experiment with setting to ZoneSetUNIV. */ +/* ArenaDefaultZONESET is the zone set used by LocusPrefDEFAULT. + * + * TODO: This is left over from before branches 2014-01-29/mps-chain-zones + * and 2014-01-17/cbs-tract-alloc reformed allocation, and may now be + * doing more harm than good. Experiment with setting to ZoneSetUNIV. */ + +#define ArenaDefaultZONESET (ZoneSetUNIV << (MPS_WORD_WIDTH / 2)) + +/* LocusPrefDEFAULT is the allocation preference used by manual pool + * classes (these don't care where they allocate). */ #define LocusPrefDEFAULT { \ LocusPrefSig, /* sig */ \