RFR: 8075416: Cleanup GC include dependencies in memoryPool.hpp

Stefan Karlsson stefan.karlsson at oracle.com
Wed Mar 18 12:16:20 UTC 2015


On 18/03/15 13:12, Mikael Gerdin wrote:
> Hi Stefan,
>
> On 2015-03-18 10:55, 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/
>
> Looks good. Reviewed.

Thanks, Mikael.

StefanK

>
> /Mikael
>
>> https://bugs.openjdk.java.net/browse/JDK-8075416
>>
>> Tested with the java/lang/management jtreg testes.
>>
>> Thanks,
>> StefanK



More information about the hotspot-dev mailing list