P5
G1SegmentedArraySegment is an object where arbitrarily sized payload data is inlined into the object.
There is a `_bottom` pointer in the class that points to this payload.
Typically in hotspot gc we tend to use some `sometype _data[1]` or so member to point to that (arbitrarily sized) data.
It would be preferable to do that here as well.
Further, DEFAULT_CACHE_LINE_SIZE alignment is not necessary for the payload - just regular alignment provided by malloc/new is sufficient (iirc). Maybe such a change could be investigated too.
Maybe it's not worth doing as the payload is typically fairly large (is it?) anyway.
Note: We use memset on the payload which creates the need for the alignment on payload, however this doesn't have to be DEFAULT_CACHE_LINE_SIZE alignment.
There is a `_bottom` pointer in the class that points to this payload.
Typically in hotspot gc we tend to use some `sometype _data[1]` or so member to point to that (arbitrarily sized) data.
It would be preferable to do that here as well.
Further, DEFAULT_CACHE_LINE_SIZE alignment is not necessary for the payload - just regular alignment provided by malloc/new is sufficient (iirc). Maybe such a change could be investigated too.
Maybe it's not worth doing as the payload is typically fairly large (is it?) anyway.
Note: We use memset on the payload which creates the need for the alignment on payload, however this doesn't have to be DEFAULT_CACHE_LINE_SIZE alignment.