RFR: 8343430: RISC-V: C2: Remove old trampoline call [v2]
    Fei Yang 
    fyang at openjdk.org
       
    Tue Nov  5 02:35:28 UTC 2024
    
    
  
On Mon, 4 Nov 2024 13:33:50 GMT, Robbin Ehn <rehn at openjdk.org> wrote:
>> Fei Yang has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Comment
>
> src/hotspot/cpu/riscv/macroAssembler_riscv.hpp line 1597:
> 
>> 1595: 
>> 1596:   enum NativeFarCall {
>> 1597:     trampoline_size        = 1 * instruction_size + wordSize,
> 
> Call address stub size?
I am a bit worried that `MacroAssembler::NativeShortCall` could be confusing with `MacroAssembler::far_call` in naming, so I simply removed this enum. I also improved the code comment about different types of calls. Let me know.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/21822#discussion_r1828646165
    
    
More information about the hotspot-dev
mailing list