RFR 8211272: x86_32 build failures after JDK-8210764 (Update avx512 implementation)
Aleksey Shipilev
shade at redhat.com
Fri Sep 28 11:16:50 UTC 2018
Bug:
https://bugs.openjdk.java.net/browse/JDK-8211272
It is a trivial mistake: the braces got unbalanced when _LP64 is not defined, which is the case for
x86_32 build. I would not bother to run it through jdk-submit.
Fix:
diff -r eb3e72f181af src/hotspot/cpu/x86/c1_LIRAssembler_x86.cpp
--- a/src/hotspot/cpu/x86/c1_LIRAssembler_x86.cpp Thu Sep 27 10:24:12 2018 +0200
+++ b/src/hotspot/cpu/x86/c1_LIRAssembler_x86.cpp Fri Sep 28 13:08:20 2018 +0200
@@ -2401,12 +2401,13 @@
{
#ifdef _LP64
if (UseAVX > 2 && !VM_Version::supports_avx512vl()) {
assert(tmp->is_valid(), "need temporary");
__ vpandn(dest->as_xmm_double_reg(), tmp->as_xmm_double_reg(),
value->as_xmm_double_reg(), 2);
- } else {
+ } else
#endif
+ {
if (dest->as_xmm_double_reg() != value->as_xmm_double_reg()) {
__ movdbl(dest->as_xmm_double_reg(), value->as_xmm_double_reg());
}
assert(!tmp->is_valid(), "do not need temporary");
__ andpd(dest->as_xmm_double_reg(),
Testing: x86_32 build, x86_64 build
Thanks,
-Aleksey
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20180928/3a2670db/signature.asc>
More information about the hotspot-compiler-dev
mailing list