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