RFR: 8075416: Cleanup GC include dependencies in memoryPool.hpp
Stefan Karlsson
stefan.karlsson at oracle.com
Wed Mar 18 10:34:16 UTC 2015
On 18/03/15 11:30, Erik Helin wrote:
> On 2015-03-18, Stefan Karlsson wrote:
>> Hi,
>>
>> Please review this patch to clean up the GC include dependencies in
>> memoryPool.hpp.
>>
>> The memoryPool.hpp header file uses GC specific code and has to include
>> header and inline files for different Spaces and Generations. I propose that
>> we decouple the memoryPool.hpp from the GC specific header files by moving
>> the definition of the offending functions, and the includes, to
>> memoryPool.cpp.
>>
>> http://cr.openjdk.java.net/~stefank/8075416/webrev.01/
>> https://bugs.openjdk.java.net/browse/JDK-8075416
> Looks good, Reviewed.
Thanks, Erik.
StefanK
>
> Thanks,
> Erik
>
>> Tested with the java/lang/management jtreg testes.
>>
>> Thanks,
>> StefanK
More information about the hotspot-dev
mailing list