RFR: ZGC: C2: Add MaxVectorSize support in C2 load barrier ad rules

Nils Eliasson nils.eliasson at oracle.com
Thu Mar 8 21:19:36 UTC 2018


Hi,

This patch fixes failures when MaxVectorSize and the size of supported 
vectors doesn't correlate. It can be easily be reproduced with: "java 
-XX:+UseZGC -XX:MaxVectorSize=16" on any CPU with AVX2 support.

"assert((uint)_ideal_reg < (uint)_last_machine_leaf && 
Type::mreg2type[_ideal_reg]) failed: in bounds)"

This bug is a big problem on many AMD systems (family < 0x17h) where 
MaxVectorSize is set default 16 even when larger vectors are supported. 
MaxVectorSize will limit the initialized vector types (TypeVect::VECTZ 
etc.) to those actually available. In the loadbarrier matcher we fail 
when we try to kill YMM but no YMM-type is initialized.

In this patch we instead have rules specialized on the number of vector 
regs that need to be killed, and only kill the lowest sub reg of the 
vector. This has the effect that the entire vector use is killed, but 
that only types that are guaranteed to be supported are available.

This patch was forgotten in a dev branch when we transfered to openJDK. 
It has been tested on the few different CPU familys available to me.

http://cr.openjdk.java.net/~neliasso/zgc/maxvectorsize

Regards,

Nils



More information about the zgc-dev mailing list