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

Erik Helin erik.helin at oracle.com
Wed Mar 18 10:30:00 UTC 2015


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

> Tested with the java/lang/management jtreg testes.
> 
> Thanks,
> StefanK


More information about the hotspot-dev mailing list