RFR: 8293351: Add second tmp register to aarch64 BarrierSetAssembler::load_at [v2]

Axel Boldt-Christmas aboldtch at openjdk.org
Mon Sep 5 12:49:15 UTC 2022


On Mon, 5 Sep 2022 12:26:53 GMT, Fei Yang <fyang at openjdk.org> wrote:

>> Axel Boldt-Christmas has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Fix argument names
>
> src/hotspot/cpu/aarch64/macroAssembler_aarch64.cpp line 2466:
> 
>> 2464: #endif
>> 2465: 
>> 2466: void MacroAssembler::resolve_jobject(Register value, Register tmp1, Register tmp2) {
> 
> You might want to rename the second parameter for its declaration in file macroAssembler_aarch64.hpp

Good catch 👍 . Fixed

-------------

PR: https://git.openjdk.org/jdk/pull/10161


More information about the shenandoah-dev mailing list