RFR: 8313438: [s390x] build broken after JDK-8301996 [v2]

Amit Kumar amitkumar at openjdk.org
Fri Oct 6 13:00:16 UTC 2023


On Fri, 6 Oct 2023 12:25:15 GMT, Martin Doerr <mdoerr at openjdk.org> wrote:

>> If you want that this case to be handled in separate PR, then I'll file a JBS issue.
>
> I need an explanation why this code was changed this way. If it is required for this PR, please explain in comments. If it's a separate issue, please handle separately.

Oh, Issue was with the `flag` register value. We need to check the volatility at the end of `putfield_or_static` method. And previous implementation was not preserving the value till the end, why it was working then ? That I don't know. 

Current Implementation uses `z_tmp_1` for saving `flag` value till end. `obj` now points to `Z_ARG5` which is also pointed by `oopStore_tmp2`, and if we don't use `agr` then in case of `object/atos` , will run into `Multiple usecase for Z_R6` error from `do_oop_store()` method .

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

PR Review Comment: https://git.openjdk.org/jdk/pull/15885#discussion_r1348689521


More information about the hotspot-compiler-dev mailing list