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

Jiangli Zhou jiangli.zhou at oracle.com
Fri Oct 26 03:51:36 UTC 2018


Thanks, David!

Jiangli


On 10/25/18 8:34 PM, David Holmes wrote:
> 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