On Tue, 11 Apr 2023 18:07:41 GMT, Martin Doerr <mdo...@openjdk.org> wrote:

> Another remark: Old JDK on s390 used "os.arch = zArch_64", current one 
> "os.arch = s390x". @offamitkumar: You probably want to take a look.

Martin, only concern was that I didn't have a good experience with `s390x` 
string in 
[past](https://github.com/openjdk/jdk/pull/13228#issuecomment-1488599607).  
But, I assume that `default` in `mapArchString` will unintentionally handle 
that scenario here. I tested it, and 'ArchTest.java' also passes.

 I guess @RealLucy might have some pointer over this.

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

PR Comment: https://git.openjdk.org/jdk/pull/13357#issuecomment-1504542573

Reply via email to