[9] RFR 8131778: java disables UseAES flag when using VIS=2 on sparc

[Konstantin Shefov]

Hi Vladimir,

Thanks for reviewing.

I have changed the webrev according to your comments: 
http://cr.openjdk.java.net/~kshefov/8131778/webrev.01

-Konstantin

On 10/30/2015 12:00 PM, Vladimir Kozlov wrote:
> sparc:
>
> Indent is wrong for !UseAES } else { block.
>
> sparc and x86: you need to set flags to false in addition to warnings:
>
>     } else if (UseAES || UseAESIntrinsics) {
> +     if (UseAES && !FLAG_IS_DEFAULT(UseAES)) {
>         warning("AES instructions are not available on this CPU");
>       }
> !     if (UseAESIntrinsics && !FLAG_IS_DEFAULT(UseAESIntrinsics)) {
> !       warning("AES intrinsics are not available on this CPU");
>       }
>
> Thanks,
> Vladimir
>
> On 10/29/15 11:43 PM, Konstantin Shefov wrote:
>> This is the fix of the issue when both flags "useAES" and
>> "useAESIntrinsics" are disabled, although only "useAESIntrinsics" should
>> be disabled if some instruction sets are not supported by a CPU.
>>
>> This fix has been tested by the slightly modified tests from
>> https://bugs.openjdk.java.net/browse/JDK-8079667 (which will be resolved
>> in the nearest future).
>>
>> -Konstantin
>>
>> On 10/27/2015 08:09 PM, Konstantin Shefov wrote:
>>> Hello
>>>
>>> Please review a bug fix.
>>>
>>> JBS entry: https://bugs.openjdk.java.net/browse/JDK-8131778
>>> Webrev: http://cr.openjdk.java.net/~kshefov/8131778/webrev.00
>>>
>>> Thanks
>>> -Konstantin
>>