CR for RFR 8164989

[Vladimir Kozlov]

Looks good. I will sponsor it.

Thanks,
Vladimir

On 9/2/16 1:05 PM, Berg, Michael C wrote:
> 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
>>