RFR(S): 8191863: [s390] Fix CDS: some bytecode rewriting doesn't depend on RewriteControl

[Lindenmaier, Goetz]

Hi Volker, 

change looks good, thanks for fixing!

Best regards,
  Goetz.

> -----Original Message-----
> From: s390x-port-dev [mailto:s390x-port-dev-bounces at openjdk.java.net]
> On Behalf Of Volker Simonis
> Sent: Montag, 27. November 2017 16:55
> To: HotSpot Open Source Developers <hotspot-dev at openjdk.java.net>
> Cc: s390x-port-dev at openjdk.java.net
> Subject: RFR(S): 8191863: [s390] Fix CDS: some bytecode rewriting doesn't
> depend on RewriteControl
> 
> Hi,
> 
> can I please have a review for the following small, s390-only change
> which fixes CDS on s390:
> 
> http://cr.openjdk.java.net/~simonis/webrevs/2017/8191863/
> https://bugs.openjdk.java.net/browse/JDK-8191863
> 
> The problem was that mistakenly rewrote zputfield into fast_zputfield
> although the active RewriteControl instance indicated that we
> shouldn't rewrite bytecodes.
> 
> With this change, all the CDS and upcoming AppCDS jtreg pass on Linux/s390.
> 
> Thank you and best regards,
> Volker