RFR (XS) 8214426 [lworld][C1] aaload doesn't work with constant index
Tobias Hartmann
tobias.hartmann at oracle.com
Wed Nov 28 10:25:37 UTC 2018
Hi Ioi,
On 28.11.18 08:24, Ioi Lam wrote:
> https://bugs.openjdk.java.net/browse/JDK-8214426
> http://cr.openjdk.java.net/~iklam/valhalla/8214426-aaload-const-index-crash.v01/
>
> The fix is to properly move the constant index into a register.
But if the index is constant, we can statically fold the following shift_left and use a LIR_Const
for the index_op, right?
> With this fix, all tests in TestBasicFunctionality passed for C1:
Great!
Best regards,
Tobias
More information about the valhalla-dev
mailing list