CR for RFR 8164989

[Berg, Michael C]

Vladimir, please see the latest webrev for the comment addition:

http://cr.openjdk.java.net/~mcberg/8164989/webrev.02/

Also I have create a new bug and referenced it to this jbs issue (https://bugs.openjdk.java.net/browse/JDK-8164989 ).

See https://bugs.openjdk.java.net/browse/JDK-8165287 for details concerning the remaining issue for compress.

Thanks,
Michael


-----Original Message-----
From: Vladimir Kozlov [mailto:vladimir.kozlov at oracle.com] 
Sent: Thursday, September 01, 2016 11:56 AM
To: hotspot-compiler-dev at openjdk.java.net; Berg, Michael C <michael.c.berg at intel.com>
Subject: Re: CR for RFR 8164989

Hi, Michael

Please, add comment which explain why it is disabled ( 0 && ).
File a bug (if you did not do that already) which will address the compress issue and reference it in the comment.

Thanks,
Vladimir

On 8/30/16 6:30 PM, Berg, Michael C wrote:
> Hi Folks,
>
> I would like to contribute a bug fix for SKX and KNL EVEX code gen.  
> The inflate and compress intrinsics on avx512 yield incorrect results 
> and cause derby, sunflow, xml.transform and xml.validation to fail.  I 
> have disabled the avx512 context for compress as it needs some rework 
> and repaired inflate.  Please review the resultant code.
>
>
>
> This code was tested as follows: hotspot jreg, SPECjvm2008 bdw, skx, 
> knl complete with no issues.  This change addresses
> http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/4a39ee246f70 which 
> was added in early May.
>
>
> Bug-id: https://bugs.openjdk.java.net/browse/JDK-8164989
>
>
> webrev:
>
> http://cr.openjdk.java.net/~mcberg/8164989/webrev/
>
>
>
> Regards,
>
> Michael
>