On Thu, 22 Apr 2021 06:19:19 GMT, Ioi Lam <ik...@openjdk.org> wrote: >> src/hotspot/share/cds/dynamicArchive.hpp line 38: >> >>> 36: #include "utilities/resourceHash.hpp" >>> 37: >>> 38: #if INCLUDE_CDS >> >> I have to wonder who is including this file and why, if CDS is not enabled. > > E.g., jvm.cpp includes dynamicArchive.hpp, but only uses its declarations > inside INCLUDE_CDS blocks. It would be too verbose to do this in jvm.cpp > > > #if INCLUDE_CDS > #include "cds/dynamicArchive.hpp" > #endif
We routinely do that for other INCLUDE_X features. ------------- PR: https://git.openjdk.java.net/jdk/pull/3610