RFR: 8339466: Enumerate shared stubs and define static fields and names via declarations [v2]
Vladimir Kozlov
kvn at openjdk.org
Tue Sep 3 18:33:26 UTC 2024
On Tue, 3 Sep 2024 10:51:53 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:
>
> fix errors in ppc generator
src/hotspot/cpu/aarch64/sharedRuntime_aarch64.cpp line 2571:
> 2569: SafepointBlob* SharedRuntime::generate_handler_blob(sharedStubId id, address call_ptr) {
> 2570: assert((id >= sharedStubId::polling_page_vectors_safepoint_handler_id ||
> 2571: id <= sharedStubId::polling_page_return_handler_id),
This and all other similar assert checks depends on the order of stubs ID.
I think such checks should be in `sharedStubId` where the order is defined:
`sharedStubId::is_polling_page_id(id)`
src/hotspot/share/runtime/stubDeclarations.hpp line 2:
> 1: /*
> 2: * Copyright (c) 2024, 2024, Oracle and/or its affiliates. All rights reserved.
You need only one 2024 year
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20832#discussion_r1742492107
PR Review Comment: https://git.openjdk.org/jdk/pull/20832#discussion_r1742486529
More information about the hotspot-dev
mailing list