RFR: 8293351: Add second tmp register to aarch64 BarrierSetAssembler::load_at [v3]
    Axel Boldt-Christmas 
    aboldtch at openjdk.org
       
    Tue Sep  6 06:24:40 UTC 2022
    
    
  
On Tue, 6 Sep 2022 00:15:08 GMT, Hao Sun <haosun at openjdk.org> wrote:
>> Axel Boldt-Christmas has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Fix argument name
>
> src/hotspot/cpu/aarch64/gc/g1/g1BarrierSetAssembler_aarch64.hpp line 48:
> 
>> 46:                             Register pre_val,
>> 47:                             Register thread,
>> 48:                             Register tmp,
> 
> It would be better to use `tmp1` as the definition site does.
👍 Fixed. Made all two temporary register names consistent across g1BarrierSetAssembler
-------------
PR: https://git.openjdk.org/jdk/pull/10161
    
    
More information about the hotspot-dev
mailing list