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

[Martin Doerr]

On Fri, 6 Oct 2023 12:57:51 GMT, Amit Kumar <amitkumar at openjdk.org> wrote:

>> 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 .

Is the agr only used to save a register? That should be explained in the comments. I believe it's only needed for the object case, right?

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

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