RFR: 8338018: Rename ClassPrelinker to AOTConstantPoolResolver

David Holmes dholmes at openjdk.org
Thu Aug 29 05:26:20 UTC 2024


On Fri, 9 Aug 2024 00:26:27 GMT, Ioi Lam <iklam at openjdk.org> wrote:

> This is the 2nd PR for [JEP 483: Ahead-of-Time Class Loading & Linking](https://bugs.openjdk.org/browse/JDK-8315737).
> 
> A simple renaming of the `ClassPrelinker` class to `AOTConstantPoolLinker`, so that the name is consistent with new classes that will be introduced in subsequent PRs for JEP 483 (`AOTClassLinker`, `AOTLinkedClassTable`, and `AOTLinkedClassBulkLoader`).
> 
> -----
> See [here](https://bugs.openjdk.org/browse/JDK-8315737) for the sequence of dependent RFEs for implementing JEP 483.

Rename looks fine, but there seems to be an unrelated change.

Thanks

src/hotspot/share/cds/aotConstantPoolResolver.hpp line 42:

> 40: class Klass;
> 41: 
> 42: template <typename T> class GrowableArray;

This doesn't seem part of the rename.

-------------

PR Review: https://git.openjdk.org/jdk/pull/20517#pullrequestreview-2267705000
PR Review Comment: https://git.openjdk.org/jdk/pull/20517#discussion_r1735584155


More information about the hotspot-dev mailing list