RFR (XXS): 8133042: Remove some direct accesses of G1Allocator::reuse_retained_old_region() to G1CollectedHeap members

Kim Barrett kim.barrett at oracle.com
Thu Aug 6 15:24:34 UTC 2015

On Aug 6, 2015, at 10:50 AM, Thomas Schatzl <thomas.schatzl at oracle.com> wrote:
>> There is very similar code to what is being changed here in G1ArchiveAllocator::alloc_new_region,
>> later in the same file (g1Allocator.cpp).
>> Would the proposed change or the additional change to alloc_new_region permit the elimination
>> of any friend declarations from G1CollectedHeap?
>  only if some additional methods from G1CollectedHeap were made public.
> Here is a webrev doing that:
> http://cr.openjdk.java.net/~tschatzl/8133042/webrev.1/ (full only
> because the original change has been so tiny)
> I am fine with either version.

Looks good.

One tiny nit:

 700   // It dirties the cards that cover the block so that so that the post

"so that so that" => "so that"


More information about the hotspot-gc-dev mailing list