RFR (M): 8154974: AVX-512 equipped inflate, has_negatives & compress intrinsics

[Vladimir Kozlov]

Thanks, Tomek

Please, add this as comment to JDK-8154974.
Still need updated webrev.

Thanks,
Vladimir

On 5/4/16 4:12 PM, Wojtowicz, Tomasz wrote:
> Internally developed ubenchmarks for performance & similar functional using Reflections API for those 3 intrinsics on an up to 4K chunks of pseudo random initialized data to verify correctness.
> Has_negatives had a negative sentinel as a last element in an array.
> Also tailored examples for compress which would test all of the possible entry scenarios (tail present/not present, non-compressable element found in head OR tail - to make sure that there is a coverage for all of the basic blocks inside intrinsic)
>
> Everything run at least 3 times for -XX:UseAVX=1 then 2 and 3, depending on a scenario for establishing performance baseline AND/OR correctness when "older"(AVX<=2) had been modified.
>
> --
> Tomek
>
> -----Original Message-----
> From: Vladimir Kozlov [mailto:vladimir.kozlov at oracle.com]
> Sent: Wednesday, May 4, 2016 3:58 PM
> To: Wojtowicz, Tomasz <tomasz.wojtowicz at intel.com>; hotspot-compiler-dev at openjdk.java.net
> Subject: Re: RFR (M): 8154974: AVX-512 equipped inflate, has_negatives & compress intrinsics
>
> Looks good. What testing was done?
>
> Thanks,
> Vladimir
>
> On 4/29/16 4:57 PM, Wojtowicz, Tomasz wrote:
>> I would like to contribute following change:
>>
>> Review details
>>
>> Review Title:      AVX-512 equipped inflate, has_negatives & compress
>> intrinsics
>>
>> Review ID:          #8154974
>>
>> Diff: http://cr.openjdk.java.net/~vdeshpande/8154974/webrev.00/
>>
>> Description:        512 bit wide upgrades for previously 128-256 wide
>> implementations using mask registers for tail computations.
>>
>> Link: https://bugs.openjdk.java.net/browse/JDK-8154974
>>
>> Author:                Tomasz, Wojtowicz
>>
>> --
>>
>> Thank you,
>>
>> Tomek
>>