RFR: 8214512: Jtreg test compiler/c2/Test8062950.java fails on ARM32

Edward Nevill edward.nevill at gmail.com
Wed Dec 12 10:55:48 UTC 2018


Hi,

Change looks fine to me, but I am not a *Reviewer*.

All the best,
Ed.

On Tue, 2018-12-11 at 17:30 -0800, dean.long at oracle.com wrote:
> I can help you commit it, but first I think another review would be
> good.
> 
> dl
> 
> On 12/11/18 5:15 PM, Nick Gasson (Arm Technology China) wrote:
> > > Hi Nick.  This change looks good to me.
> > 
> > Thanks Dean. Could you help me to commit this patch if it doesn't
> > need other reviews?
> > 
> > Nick
> > 
> > 
> > > -----Original Message-----
> > > From: dean.long at oracle.com <dean.long at oracle.com>
> > > Sent: 12 December 2018 06:05
> > > To: Nick Gasson (Arm Technology China) <Nick.Gasson at arm.com>;
> > > hotspot-
> > > dev at openjdk.java.net; Boris Ulasevich <boris.ulasevich at bell-sw.co
> > > m>
> > > Cc: nd <nd at arm.com>; hotspot-compiler-dev at openjdk.java.net
> > > compiler
> > > <hotspot-compiler-dev at openjdk.java.net>; aarch32-port-
> > > dev at openjdk.java.net
> > > Subject: Re: RFR: 8214512: Jtreg test
> > > compiler/c2/Test8062950.java fails on
> > > ARM32
> > > 
> > > Hi Nick.  This change looks good to me.
> > > 
> > > dl
> > > 
> > > On 12/2/18 10:06 PM, Nick Gasson wrote:
> > > > Hi,
> > > > 
> > > > Could someone please help me review this fix for a test failure
> > > > on ARM32:
> > > > 
> > > > Bug: https://bugs.openjdk.java.net/browse/JDK-8214512
> > > > Webrev: http://cr.openjdk.java.net/~njian/8214512/webrev.0/
> > > > 


More information about the aarch32-port-dev mailing list