RFR: 8047820: G1 Block offset table does not need to support generic Space classes

[Stefan Karlsson]

On 2014-06-25 13:25, Mikael Gerdin wrote:
> Hi!
>
> On Monday 23 June 2014 16.25.53 Mikael Gerdin wrote:
>> Hi!
>>
>> As part of a larger effort to detach G1's HeapRegion from ContiguousSpace[1]
>> G1's block offset table needs to be modified to work with Space subclasses
>> which are not subclasses of ContiguousSpace. Just change the code to have
>> knowledge of G1OffsetTableContigSpace.
>>
>> This change is part of a set of 4 changes: 8047818, 8047819, 8047820,
>> 8047821 which are needed to refactor the HeapRegion class and its
>> superclasses in order to simplify the G1 class unloading change which is
>> coming.
>>
>> Bug:
>> https://bugs.openjdk.java.net/browse/JDK-8047820
>> Webrev:
>> http://cr.openjdk.java.net/~mgerdin/8047820/webrev/
> I discovered that I accidentally put the set_offset_array change in the
> 8047818 webrev. It is actually needed in this change to make the JVM compile
> without precompiled headers.
>
> Here's the new full webrev:
> http://cr.openjdk.java.net/~mgerdin/8047820/webrev.2/
>
> Incremental webrev:
> http://cr.openjdk.java.net/~mgerdin/8047820/webrev.1_to_2/

Looks good.

StefanK

>
> Thanks
> /Mikael
>
>> [1] https://bugs.openjdk.java.net/browse/JDK-8047818
>>
>> Thanks
>> /Mikael