Integrated: 8337788: RISC-V: Cleanup code in MacroAssembler::reserved_stack_check
    Gui Cao 
    gcao at openjdk.org
       
    Tue Aug  6 02:19:40 UTC 2024
    
    
  
On Sun, 4 Aug 2024 02:43:26 GMT, Gui Cao <gcao at openjdk.org> wrote:
> Hi, 
> In the MacroAssembler::reserved_stack_check() function:
> 
>     RuntimeAddress target(StubRoutines::throw_delayed_StackOverflowError_entry());
>     relocate(target.rspec(), [&] {
>       int32_t offset;
>       movptr(t0, target.target(), offset);
>       jr(t0, offset);
>     });
> 
> can be simplified to:
> 
>   la(t0, RuntimeAddress(StubRoutines::throw_delayed_StackOverflowError_entry()));
>   jr(t0);
> 
> 
> In addition, the code formatting has been modified to remove the extra spaces before the code.
> 
> Please take a look and have some reviews. Thanks a lot.
> 
> ### Testing
> - [x] Run tier1-3 tests on SOPHON SG2042 (fastdebug)
This pull request has now been integrated.
Changeset: 73718fb8
Author:    Gui Cao <gcao at openjdk.org>
Committer: Fei Yang <fyang at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/73718fb8a3570023e7855137eb008f78b8a1e8ce
Stats:     19 lines in 1 file changed: 0 ins; 4 del; 15 mod
8337788: RISC-V: Cleanup code in MacroAssembler::reserved_stack_check
Reviewed-by: fyang, rehn, fjiang
-------------
PR: https://git.openjdk.org/jdk/pull/20458
    
    
More information about the hotspot-dev
mailing list