[10] RFR (XS) 8182114: assert(VM_Version::supports_sse()) failed

[Vladimir Kozlov]

Thank you, Dan

Vladimir

On 6/15/17 1:35 PM, Daniel D. Daugherty wrote:
> On 6/15/17 1:20 PM, Vladimir Kozlov wrote:
>> CCing to runtime since the failure happened in Interpreter in FMA code.
>>
>> It is regression after JDK-8181616 change which replaced UseSSE >= 2 
>> with supports_avx().
>>
>> Restore UseSSE check.
>>
>> http://cr.openjdk.java.net/~kvn/8182114/webrev/
> 
> src/cpu/x86/vm/vm_version_x86.cpp
>      L815:   if (supports_fma() && UseSSE >= 2) { // Check UseSSE since 
> FMA code uses SSE instructions
>          Agreed that restoring the "&& UseSSE >= 2" part to the
>          if-statement is the (easiest and) right solution here.
> 
> Thanks for the nice analysis in the bug report.
> 
> Thumbs up.
> 
> Dan
> 
> 
>> https://bugs.openjdk.java.net/browse/JDK-8182114
>>
>> RBT testing is running.
>>
>> Thanks,
>> Vladimir
>>
>