RFR(S): 8210381: Obsolete EmitSync

[Baesken, Matthias]

Hello, small update from my side - builds were fine with the  patch included   on   our   ppc64(le) / s390x  platforms .
I did not notice any  test failures related to the patch .


Best regards, Matthias


> -----Original Message-----
> From: Baesken, Matthias
> Sent: Mittwoch, 5. September 2018 13:09
> To: 'Mikael Vidstedt' <mikael.vidstedt at oracle.com>;
> daniel.daugherty at oracle.com; Doerr, Martin <martin.doerr at sap.com>
> Cc: s390x-port-dev at openjdk.java.net; ppc-aix-port-dev at openjdk.java.net;
> HotSpot Open Source Developers <hotspot-dev at openjdk.java.net>;
> aarch64-port-dev at openjdk.java.net
> Subject: RE: RFR(S): 8210381: Obsolete EmitSync
> 
> Hello  ,   I had a short look  at the ppc / s390x changes ,   looks OK .
> To be on the safe side I put  it into  our  build/test queue,  so we see results
> for ppc64(le) / s390x as well tomorrow .
> 
> Best regards, Matthias
> 
> > -----Original Message-----
> > From: ppc-aix-port-dev <ppc-aix-port-dev-bounces at openjdk.java.net>
> On
> > Behalf Of Mikael Vidstedt
> > Sent: Mittwoch, 5. September 2018 03:38
> > To: daniel.daugherty at oracle.com
> > Cc: s390x-port-dev at openjdk.java.net; ppc-aix-port-
> dev at openjdk.java.net;
> > HotSpot Open Source Developers <hotspot-dev at openjdk.java.net>;
> > aarch64-port-dev at openjdk.java.net
> > Subject: Re: RFR(S): 8210381: Obsolete EmitSync
> >
> >
> > Dan/Vladimir, thanks for the reviews.
> >
> > FWIW, in addition to the build&test job I also did some manual work to
> > compare the disassembly of the resulting libjvm; the C++ compiler seems
> to
> > agree with my “manual” dead code elimination (yay!).
> >
> > Cheers,
> > Mikael