[aarch64-port-dev ] [RFR 8u]: 8224851: AArch64: fix warnings and errors with Clang and GCC 8.3

Severin Gehwolf sgehwolf at redhat.com
Wed Mar 11 12:21:47 UTC 2020


On Fri, 2020-03-06 at 14:02 +0000, Andrew Haley wrote:
> On 3/6/20 1:36 PM, Severin Gehwolf wrote:
> > While we could backport only the relevant hunk to fix the GCC 10 issue
> > I've tried a more complete backport as it seems to be fixing a couple
> > of code issues. If it's preferred, I can do a select backport of [1]
> > too. Thoughts?
> > 
> > Bug: https://bugs.openjdk.java.net/browse/JDK-8224851
> > webrev: http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8224851/01/webrev/
> > JDK 11u change: https://hg.openjdk.java.net/jdk-updates/jdk11u/rev/15cc1c8a6371
> 
> This webrev looks very close to what I'd do. The fixes you've made
> look correct, and in some cases are real bugs which impact code
> generation.
> 
> OK.

Thanks for the review!

If somebody could sponsor this patch for me I'd appreciate it. I'm not
a committer for the aarch64-port project.

HG exported patch is here:
https://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8224851/JDK-8224851.aarch64.export.patch

Thanks,
Severin



More information about the aarch64-port-dev mailing list