RFR (M): 8146581: Minor corrections to the patch submitted for earlier bug id - 8143925

[Kharbas, Kishor]

Wow! That was quick.. thanks :)

-----Original Message-----
From: Vladimir Kozlov [mailto:vladimir.kozlov at oracle.com] 
Sent: Thursday, January 07, 2016 2:10 PM
To: Kharbas, Kishor; hotspot-compiler-dev at openjdk.java.net
Subject: Re: RFR (M): 8146581: Minor corrections to the patch submitted for earlier bug id - 8143925

Looks good.

Thanks,
Vladimir

On 1/7/16 2:05 PM, Kharbas, Kishor wrote:
> Hi Vladimir,
>
> For the CounterMode.crypt() patch 
> (https://bugs.openjdk.java.net/browse/JDK-8143925) which was committed earlier, I found a minor correction in the checks performed for AES support in vm_version_x86.cpp.
>
> Basically, a condition check was missing in a else if() block, and some old code was left by accident.
>
> I also took this opportunity to add some more comments to make the stub code more readable/maintainable.
>
> Bug - https://bugs.openjdk.java.net/browse/JDK-8146581
>
> Patch - http://cr.openjdk.java.net/~vdeshpande/8146581/webrev.00/
>
> Regards
>
> Kishor Kharbas
>