RFR: 8366475: Rename MetaspaceShared class to AOTMetaspace [v4]

Ashutosh Mehra asmehra at openjdk.org
Thu Sep 4 14:01:51 UTC 2025


On Thu, 4 Sep 2025 05:38:20 GMT, Ioi Lam <iklam at openjdk.org> wrote:

>> This PR is one (of many) steps in [JDK-8366473](https://bugs.openjdk.org/browse/JDK-8366473) (Refactor CDS source code with new AOT terminology):
>> 
>> Rename this class to with a naming convention that's consistent with other new AOT classes such as AOTClassLinker.
>> 
>> All changes are mechanical text replacement. Headers are resorted alphabetically.
>
> Ioi Lam has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 11 commits:
> 
>  - Fixed include order
>  - Merge branch 'master' into 8366475-rename-metaspace-shared-to-aot-metaspace
>  - Merge branch '8366474-rename_metaspaceobj_is_shared_to_in_aot_cache' into 8366475-rename-metaspace-shared-to-aot-metaspace
>  - Also renamed ConstantPool::is_shared() because it shadows MetaspaceObj::is_shared()
>  - Merge branch 'master' into 8366474-rename_metaspaceobj_is_shared_to_in_aot_cache
>  - removed unnecessary includes
>  - Redo changes in comments and tests
>  - Merge branch '8366474-rename_metaspaceobj_is_shared_to_in_aot_cache' into 8366475-rename-metaspace-shared-to-aot-metaspace
>  - Removed MetaspaceShared -> AOTMetaspace changes that are intended for https://github.com/openjdk/jdk/pull/27017
>  - Rename MetaspaceShared -> AOTMetaspace
>  - ... and 1 more: https://git.openjdk.org/jdk/compare/90a2db1e...63073feb

lgtm

@iklam thanks for addressing the comments.

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

Marked as reviewed by asmehra (Committer).

PR Review: https://git.openjdk.org/jdk/pull/27017#pullrequestreview-3185475117
PR Comment: https://git.openjdk.org/jdk/pull/27017#issuecomment-3253872538


More information about the hotspot-dev mailing list