Hi, can I consider this reviewed? Remi Forax and Phil Race had a look at this.
Best regards, Goetz. > -----Original Message----- > From: Lindenmaier, Goetz > Sent: Montag, 4. Juli 2016 08:33 > To: 'Phil Race' <philip.r...@oracle.com>; awt-...@openjdk.java.net; 2d-dev > <2d-dev@openjdk.java.net> > Subject: RE: <AWT Dev> RFR(XS): 8160721: Avoid deoptimizations in > Font.equals. > > Hi, > > Resending this to 2d-dev. > > Also, I made a new webrev considering Remi's comments: > http://cr.openjdk.java.net/~goetz/wr16/8160721-deoptFont/webrev.02/ > > Best regards, > Goetz. > > > > -----Original Message----- > > From: Phil Race [mailto:philip.r...@oracle.com] > > Sent: Freitag, 1. Juli 2016 19:35 > > To: Lindenmaier, Goetz <goetz.lindenma...@sap.com>; awt- > > d...@openjdk.java.net > > Subject: Re: <AWT Dev> RFR(XS): 8160721: Avoid deoptimizations in > > Font.equals. > > > > Please resend this to 2d-dev. All font related API and implementation > > is part of 2D, not AWT. > > > > -phil. > > > > On 7/1/2016 6:59 AM, Lindenmaier, Goetz wrote: > > > > > > Hi, > > > > > > This small fix improves performance of the C2 compiler. > > > > > > We see deoptimizations due to class casts in Font.equals. > > > Use instanceof check instead of catching the exception. > > > > > > Please review this change: > > > > > > http://cr.openjdk.java.net/~goetz/wr16/8160721-deoptFont/ > > > <http://cr.openjdk.java.net/%7Egoetz/wr16/8160721-deoptFont/> > > > > > > Best regards, > > > > > > Goetz. > > >
