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 shenandoah-dev mailing list