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

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 INCLUDECDS in classLoaderExt.cpp and classLoaderExt.hpp.

- Removed #if INCLUDECDS from classLoaderExt.cpp since the file is not included when CDS feature is excluded. - Cleaned up classLoaderExt.hpp and moved CDSONLY functions & enum under #if INCLUDECDS. 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