RFR(S): 8210381: Obsolete EmitSync

[Mikael Vidstedt]

Thanks a lot for all the reviews and for the help verifying the change across the platforms!

Cheers,
Mikael

> On Sep 5, 2018, at 11:56 PM, Baesken, Matthias <matthias.baesken at sap.com> wrote:
> 
> 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
>