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' <[email protected]>; [email protected]; 2d-dev > <[email protected]> > 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:[email protected]] > > Sent: Freitag, 1. Juli 2016 19:35 > > To: Lindenmaier, Goetz <[email protected]>; awt- > > [email protected] > > 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. > > >
