RFR: 8334060: Implementation of Late Barrier Expansion for G1 [v17]
Roberto Castañeda Lozano
rcastanedalo at openjdk.org
Mon Sep 9 11:48:11 UTC 2024
On Fri, 6 Sep 2024 23:57:59 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:
>> Roberto Castañeda Lozano has updated the pull request incrementally with one additional commit since the last revision:
>>
>> s390 port : late barrier expansion
>
> src/hotspot/cpu/x86/gc/g1/g1BarrierSetAssembler_x86.cpp line 354:
>
>> 352: __ bind(runtime);
>> 353: // save the live input values
>> 354: RegSet saved = RegSet::of(store_addr NOT_LP64(COMMA thread));
>
> I was looking at this a while ago, and haven't figured out why we're saving `store_addr` here.
> Also not sure why we're saving `thread` here for 32bit platforms.
> Something to think about for the future. Though maybe the 32bit case will be gone by then :)
I'm not sure either, this is in any case pre-existing interpreter code.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/19746#discussion_r1750105760
More information about the hotspot-dev
mailing list