RFR(s): 8079330: Circular dependency between G1CollectedHeap and G1BlockOffsetSharedArray

Per Liden per.liden at oracle.com
Tue May 5 11:57:48 UTC 2015


On 2015-05-05 13:17, Mikael Gerdin wrote:
> Hi Per,
>
> On 2015-05-05 12:11, Per Liden wrote:
>> There's a circular include dependency between
>> g1BlockOffsetTable.inline.hpp and g1CollectedHeap.inline.hpp. While
>> working on a different change, which sorted the include lines in
>> g1CollectedHeap.inline.hpp, this dependency was unveiled and caused a
>> build error.
>>
>> This patch turns the check_index() macro in
>> g1BlockOffsetTable.inline.hpp into a function and moves it to
>> g1BlockOffsetTable.cpp.
>>
>> Webrev: http://cr.openjdk.java.net/~pliden/8079330/webrev.0/
>
> Looks good.
> Thanks for untangling this mess a bit.
> /m

Thanks for reviewing Mikael.

/Per



More information about the hotspot-gc-dev mailing list