[aarch64-port-dev ] jdk1 Fix for hotspot/test/gc/g1/TestJNIWeakG1/TestJNIWeakG1.java

[Stuart Monteith]

Thanks Felix!

The patch cleanly applies to jdk9/dev, and the issue that was
identified also exists on  However, the JTReg testcase that identified
the issue does not exist on jdk9. I'm currently running the existing
JTReg tests against a jdk9/dev build with the patch.

BR,
   Stuart


On 30 August 2017 at 14:02, Felix Yang <felix.yang at linaro.org> wrote:
> Looks good. Pushed.
> Shall we apply this patch to the jdk9 dev repo?
>
> Thanks,
> Felix
>
> On 30 August 2017 at 18:37, Stuart Monteith <stuart.monteith at linaro.org>
> wrote:
>>
>> Thanks for your reviews Andrew!
>>
>>
>> On 29 August 2017 at 13:15, Andrew Dinn <adinn at redhat.com> wrote:
>> > On 29/08/17 11:10, Stuart Monteith wrote:
>> >> Hello,
>> >>    Ningsheng has put my latest patch up for review. I've backed out
>> >> passing the tmp registers as parameters, and simply checking them in
>> >> the asserts.  g1_write_barrier_post was also being passed a
>> >> conflicting register, so that was fixe d up too:
>> >>
>> >>  http://cr.openjdk.java.net/~njian/8186325/webrev.01/
>> > Yes, that looks ok to push.
>> >
>> > regards,
>> >
>> >
>> > Andrew Dinn
>> > -----------
>> > Senior Principal Software Engineer
>> > Red Hat UK Ltd
>> > Registered in England and Wales under Company Registration No. 03798903
>> > Directors: Michael Cunningham, Michael ("Mike") O'Neill, Eric Shander
>
>