On Fri, 23 Oct 2026 04:01:31 GMT, David Holmes <[email protected]> wrote:
>> The C++ class `SystemDictionaryShared` should be renamed to >> `AOTSystemDictionary`, to be consistent with the rest of the AOT code. >> >> This RFE changes only class names. Further changes are needed to rename the >> various `xxx_shared_xxx` symbols in the C++ code. >> >> >> --------- >> - [x] I confirm that I make this contribution in accordance with the >> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai). > > make/hotspot/lib/JvmFeatures.gmk line 124: > >> 122: JVM_EXCLUDE_FILES += \ >> 123: aotCodeCache.cpp \ >> 124: aotCompileBroker.cpp \ > > So are there not AOT specific files that should be excluded in place of these? > > We have "cds" as a JVM-feature - is "aot" also a JVM-feature? "AOT" is our new term for "CDS", so they are equivalent. The goal of the parent issue ([JDK-8366473](https://bugs.openjdk.org/browse/JDK-8366473)) is to replace all uses of "CDS" in the source code to "AOT". This PR is just one of the steps. In the near future, "CDS" will still appear as a user visible feature, so this term will remain in documentation as well as some literal strings inside the source code. We are preparing to deprecate and remove CDS features such as the static/dynamic CDS archives. So eventually you will have only AOT caches. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/33475#discussion_r4356506442
