RFR (M): 8152354: Update for x86 AES CBC Decryption

Deshpande, Vivek R vivek.r.deshpande at intel.com
Thu Mar 24 20:00:20 UTC 2016

Hi Vladimir

Thanks for pushing it.
I will make sure that webrev works.


-----Original Message-----
From: Vladimir Kozlov [mailto:vladimir.kozlov at oracle.com] 
Sent: Thursday, March 24, 2016 12:17 PM
To: Deshpande, Vivek R <vivek.r.deshpande at intel.com>; hotspot compiler <hotspot-compiler-dev at openjdk.java.net>
Cc: Viswanathan, Sandhya <sandhya.viswanathan at intel.com>
Subject: Re: RFR (M): 8152354: Update for x86 AES CBC Decryption

Testing passed. I submitted JPRT push.

BTW, Vivek, your patch files (this and previous) contain CR at the end of lines. It looks like webrev was generated on Windows. As result the patch can't be applied and I have to clean them (dos2unix). It would be nice if you fix that in your future webrevs.


On 3/23/16 6:40 PM, Vladimir Kozlov wrote:
> Looks fine. I submitted testing. Will push if testing pass.
> Thanks,
> Vladimir
> On 3/21/16 3:18 PM, Deshpande, Vivek R wrote:
>> Hi all
>> I would like to contribute a patch which optimizesAES CBC Decryption 
>> for X86architecture using 4 way parallel version for key sizes 192 and 256 for 64 bit and for key sizes 128, 192 and 256 for 32 bit.
>> Could you please review and sponsor this patch.
>> Bug-id:
>> https://bugs.openjdk.java.net/browse/JDK-8152354
>> webrev:
>> http://cr.openjdk.java.net/~vdeshpande/AES_CBC/8152354/webrev.00/
>> Thanks and regards,
>> Vivek

More information about the hotspot-compiler-dev mailing list