RFR: 8339466: Enumerate shared stubs and define static fields and names via declarations [v4]
Andrew Dinn
adinn at openjdk.org
Thu Sep 5 10:28:41 UTC 2024
On Wed, 4 Sep 2024 22:20:31 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:
>> Andrew Dinn has updated the pull request incrementally with one additional commit since the last revision:
>>
>> clean up asserts
>
> 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`
Done here and at all other occurrences.
> src/hotspot/share/runtime/sharedRuntime.hpp line 66:
>
>> 64: #undef SHARED_STUB_FIELD_DECLARE
>> 65:
>> 66: #ifndef PRODUCT
>
> Use `#ifdef ASSERT` here.
Done
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20832#discussion_r1745204621
PR Review Comment: https://git.openjdk.org/jdk/pull/20832#discussion_r1745205133
More information about the hotspot-dev
mailing list