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