[11u] RFR: JDK-8222286: Fix for JDK-8213419 is broken on s390

[Andrew Hughes]

On 09:27 Wed 16 Sep     , Andrew Haley wrote:
> On 16/09/2020 04:05, Andrew Hughes wrote:
> > Bug: https://bugs.openjdk.java.net/browse/JDK-8222286
> > Webrev: https://cr.openjdk.java.net/~andrew/openjdk11/8222286/webrev.01/
> > 
> > On s390 (the 31-bit architecture), calls to log2_intptr with a size_t
> > argument are ambiguous, because there is log2_intptr(uintptr_t x) and
> > log2_intptr(intptr_t x). This is not a problem on other architectures
> > where size_t is the same as uintptr_t. See existing bugs for
> > more details of the s390 situation [0] [1] [2].
> > 
> > The issue can be resolved by explicitly casting to uintptr_t.
> > 
> > It is not applicable to >= 12u, as JDK-8213436 removed the problematic
> > function in which this occurs.
> > 
> > Ok for 11u?
> 
> Sure, but please add a comment. "S/390: Cast to (uintptr_t): see JDK-8222286"
> would do. Thanks.
> 
> -- 
> Andrew Haley  (he/him)
> Java Platform Lead Engineer
> Red Hat UK Ltd. <https://www.redhat.com>
> https://keybase.io/andrewhaley
> EAC8 43EB D3EF DB98 CC77 2FAD A5CD 6035 332F A671
> 

Sure, good idea.

Added in https://cr.openjdk.java.net/~andrew/openjdk11/8222286/webrev.02/

Thanks,
-- 
Andrew :)

Senior Free Java Software Engineer
OpenJDK Package Owner
Red Hat, Inc. (http://www.redhat.com)

PGP Key: ed25519/0xCFDA0F9B35964222 (hkp://keys.gnupg.net)
Fingerprint = 5132 579D D154 0ED2 3E04  C5A0 CFDA 0F9B 3596 4222