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

[Andrew Hughes]

On 11/03/2020 12:21, Severin Gehwolf wrote:
> 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
> 

Pushed.

Thanks,
-- 
Andrew :)

Senior Free Java Software Engineer
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