RFR: 8281079: [s390] Unify Address Operand Encoding in Instruction Emitters

[Lutz Schmidt]

On Mon, 21 Feb 2022 14:02:37 GMT, Martin Doerr <mdoerr at openjdk.org> wrote:

>> May I please request reviews for this cleanup and unification change. It streamlines the address operand emitters for all instructions. As such, the modifications are widespread and may be cumbersome to review. Please ask questions if anything is unclear.
>> 
>> SAP does not maintain a full s390x build and test environment any longer. I would therefore very much appreciate if somebody would run a build and test cycle on s390x. 
>> 
>> As this is a s390x-only change, other platforms are not impacted.
>
> src/hotspot/cpu/s390/assembler_s390.inline.hpp line 52:
> 
>> 50:         *(unsigned short*)(pos)   = (unsigned short)(x>>16);
>> 51:         *(unsigned short*)(pos+2) = (unsigned short)x;
>> 52:       }
> 
> Looks correct, but what is the motivation behind splitting unaligned stores? z can store to unaligned memory.

Will go back to single int store for 4-byte instructions.

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

PR: https://git.openjdk.java.net/jdk/pull/7324