RFR: 8339466: Enumerate shared stubs and define static fields and names via declarations [v4]

Vladimir Kozlov kvn at openjdk.org
Wed Sep 4 22:26:54 UTC 2024


On Wed, 4 Sep 2024 11:10:36 GMT, Andrew Dinn <adinn at openjdk.org> wrote:

>> Systematize handling of SharedRuntime stubs. Generate enum ids, static fields and names from declarations using template macros. Systematically reference stubs and stub names using ids.
>
> Andrew Dinn has updated the pull request incrementally with one additional commit since the last revision:
> 
>   clean up asserts

Few comments.

src/hotspot/cpu/aarch64/sharedRuntime_aarch64.cpp line 2184:

> 2182:   }
> 2183: #endif
> 2184:   const char *name = SharedRuntime::stub_name(SharedStubId::deopt_id);

Code style: `const char* name`

src/hotspot/share/runtime/sharedRuntime.hpp line 66:

> 64: #undef SHARED_STUB_FIELD_DECLARE
> 65: 
> 66: #ifndef PRODUCT

Use `#ifdef ASSERT` here.

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

PR Review: https://git.openjdk.org/jdk/pull/20832#pullrequestreview-2281512904
PR Review Comment: https://git.openjdk.org/jdk/pull/20832#discussion_r1744577365
PR Review Comment: https://git.openjdk.org/jdk/pull/20832#discussion_r1744579368


More information about the hotspot-dev mailing list