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

Andrew Dinn adinn at openjdk.org
Wed Sep 4 10:58:43 UTC 2024


On Wed, 4 Sep 2024 02:43:44 GMT, Fei Yang <fyang at openjdk.org> wrote:

>> Andrew Dinn has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   fix errors in ppc generator
>
> Hi Andrew:
>     I just checked riscv-specific changes and seems that we are lacking following small change. 
>     Could you please add it? I did a release build and ran tier1 test on linux-riscv64 platform. 
> 
> 
> diff --git a/src/hotspot/cpu/riscv/sharedRuntime_riscv.cpp b/src/hotspot/cpu/riscv/sharedRuntime_riscv.cpp
> index 2e8362814de..0481bc3483b 100644
> --- a/src/hotspot/cpu/riscv/sharedRuntime_riscv.cpp
> +++ b/src/hotspot/cpu/riscv/sharedRuntime_riscv.cpp
> @@ -2447,7 +2447,8 @@ SafepointBlob* SharedRuntime::generate_handler_blob(sharedStubId id, address cal
>    OopMap* map = nullptr;
> 
>    // Allocate space for the code.  Setup code generation tools.
> -  CodeBuffer buffer("handler_blob", 2048, 1024);
> +  const char *name = SharedRuntime::stub_name(id);
> +  CodeBuffer buffer(name, 2048, 1024);
>    MacroAssembler* masm = new MacroAssembler(&buffer);
>    assert_cond(masm != nullptr);
> 
> @@ -2455,7 +2456,7 @@ SafepointBlob* SharedRuntime::generate_handler_blob(sharedStubId id, address cal
>    address call_pc = nullptr;
>    int frame_size_in_words = -1;
>    bool cause_return = (id == sharedStubId::polling_page_return_handler_id);
> -  RegisterSaver reg_save(id == sharedStubId::polling_page_vectors_safepoint_handler_id /* save_vectors */);
> +  RegisterSaver reg_saver(id == sharedStubId::polling_page_vectors_safepoint_handler_id /* save_vectors */);
> 
>    // Save Integer and Float registers.
>    map = reg_saver.save_live_registers(masm, 0, &frame_size_in_words);

@RealFYang Thanks for checking the riscv code. I have pushed a new version that should fix the errors.

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

PR Comment: https://git.openjdk.org/jdk/pull/20832#issuecomment-2328575392


More information about the hotspot-dev mailing list