PR3634: Shenandoah still broken on s390 with aarch64-shenandoah-jdk8u181-b16

Andrew Hughes gnu.andrew at redhat.com
Mon Nov 12 16:34:06 UTC 2018


The build of Shenandoah is still broken on s390 (the 31-bit platform)
after the latest merge, due to a mismatch in the MAX2 macro:

https://icedtea.classpath.org/bugzilla/show_bug.cgi?id=3634

The working solution I have for this at present (and used in our
latest RPMs & IcedTea release) is to update ShenandoahMinRegionSize
and ShenandoahMaxRegionSize to be size_t instead of uintx. This
shouldn't make a difference on other platforms where they are the
same. It does on s390 where size_t is bigger than uintx.

https://cr.openjdk.java.net/~andrew/shenandoah-8/pr3634/webrev.01/

Ok to push to aarch64/shenandoah-jdk8u? And maybe elsewhere?

Thanks,
-- 
Andrew :)

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

Web Site: http://fuseyism.com
Twitter: https://twitter.com/gnu_andrew_java
PGP Key: ed25519/0xCFDA0F9B35964222 (hkp://keys.gnupg.net)
Fingerprint = 5132 579D D154 0ED2 3E04  C5A0 CFDA 0F9B 3596 4222


More information about the shenandoah-dev mailing list