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

[Daniel D. Daugherty]

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
>