RFR: 8339416: [s390x] Provide implementation for resolve_global_jobject [v2]

Martin Doerr mdoerr at openjdk.org
Mon Sep 16 16:49:07 UTC 2024


On Mon, 16 Sep 2024 04:56:36 GMT, Amit Kumar <amitkumar at openjdk.org> wrote:

>> This PR provides "resolve_global_jobject" method implementation for s390x-port. 
>> 
>> Testing:
>> * Tier1 test with Fastdebug; 
>> * Added these changes on top of https://github.com/openjdk/jdk/pull/20479 and modified the call in the stubGenerator_s390.cpp file; 
>> *  1. Ran tier1 test with a call to  "resolve_jobect"
>> *  2. Ran tier1 test with a call to  "resolve_global_jobject" 
>> 
>> I didn't see any new failure appearing there.
>
> Amit Kumar has updated the pull request incrementally with one additional commit since the last revision:
> 
>   removes extra line

Looks correct to me.

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.

src/hotspot/cpu/s390/gc/shared/barrierSetAssembler_s390.cpp line 146:

> 144: 
> 145:   __ z_ltgr(value, value);
> 146:   __ z_bre(done); // null as-is.

"Use null as-is." sounds better.

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

PR Review: https://git.openjdk.org/jdk/pull/20986#pullrequestreview-2307174155
PR Review Comment: https://git.openjdk.org/jdk/pull/20986#discussion_r1761487260
PR Review Comment: https://git.openjdk.org/jdk/pull/20986#discussion_r1761487962


More information about the hotspot-dev mailing list