RFR: 8339416: [s390x] Provide implementation for resolve_global_jobject [v2]
Martin Doerr
mdoerr at openjdk.org
Tue Sep 17 11:01:41 UTC 2024
On Tue, 17 Sep 2024 08:57:59 GMT, Amit Kumar <amitkumar at openjdk.org> wrote:
>> src/hotspot/cpu/s390/gc/shared/barrierSetAssembler_s390.cpp line 108:
>>
>>> 106: }
>>> 107:
>>> 108: // Generic implementation. GCs can provide an optimized one.
>>
>> You may want to implement an optimized `G1BarrierSetAssembler::resolve_jobject` and `ModRefBarrierSetAssembler::resolve_jobject`. Otherwise, those GCs may get a regression.
>
> I have added implementation for `ModRefBarrierSetAssembler::resolve_jobject`. It seems `G1BarrierSetAssembler::resolve_jobject` is already present, do I need to do anything extra than that ?
I think it's fine.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20986#discussion_r1762839103
More information about the hotspot-dev
mailing list