RFR (trivial) 8205327: Clean up #if INCLUDE_CDS in classLoaderExt.cpp and classLoaderExt.hpp

David Holmes david.holmes at oracle.com
Fri Oct 26 03:34:15 UTC 2018


Hi Jiangli,

This looks okay to me.

Thanks,
David

On 26/10/2018 12:11 PM, Jiangli Zhou wrote:
> Please review this trivial change that cleans up #if INCLUDE_CDS in 
> classLoaderExt.cpp and classLoaderExt.hpp.
> 
> - Removed #if INCLUDE_CDS from classLoaderExt.cpp since the file is not 
> included when CDS feature is excluded.
> - Cleaned up classLoaderExt.hpp and moved CDS_ONLY functions & enum 
> under #if INCLUDE_CDS.
> 
> webrev: http://cr.openjdk.java.net/~jiangli/8205327/webrev.00/
> RFE: https://bugs.openjdk.java.net/browse/JDK-8205327
> 
> Tested with normal and minimum builds locally on linux-x64. Running 
> tier1-tier5 tests.
> 
> Thanks,
> Jiangli


More information about the hotspot-runtime-dev mailing list