RFR: 8377554: Load card table base and other values via AOTRuntimeConstants in AOT code [v3]

Vladimir Kozlov kvn at openjdk.org
Tue Feb 24 17:01:42 UTC 2026


On Tue, 24 Feb 2026 16:27:32 GMT, Ashutosh Mehra <asmehra at openjdk.org> wrote:

>> Andrew Dinn has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   more VNKozlov feedback
>
> src/hotspot/cpu/x86/gc/g1/g1BarrierSetAssembler_x86.cpp line 308:
> 
>> 306:     __ shrptr(save);
>> 307:     __ pop(rcx);
>> 308:     __ mov(tmp1, save);
> 
> Is this `mov` redundant? `tmp1` value is not used after this.

Yes, you don't need it.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/29884#discussion_r2848392671


More information about the hotspot-dev mailing list