RFR 8076276 support for AVX512

Roland Westrelin roland.westrelin at oracle.com
Thu May 7 10:25:05 UTC 2015


> http://cr.openjdk.java.net/~kvn/8076276/webrev.02

This looks good to me. A few minor remarks:

Shouldn’t the #ifdef _LP64 new instruct be in x86_64.ad? I see there are already other #ifdef _LP64 in x86.ad so I’m not sure what the guideline is.

In vm_version_x86.hpp, os_supports_avx_vectors(), you could have a single copy of the loop with different parameters.

Not sure why you dropped:

3463 // Float register operands
3473 // Double register operands

in x86_64.ad

In chaitin.hpp:

144   uint16_t _num_regs;            // 2 for Longs and Doubles, 1 for all else

comment is not aligned with the one below anymore.

Roland.




More information about the hotspot-compiler-dev mailing list