RFR(S): 8231961: [lworld] C1 compiled code returns incorrect value for flattened array access

Tobias Hartmann tobias.hartmann at oracle.com
Wed Oct 9 12:43:56 UTC 2019


Hi,

please review the following patch:
https://bugs.openjdk.java.net/browse/JDK-8231961
http://cr.openjdk.java.net/~thartmann/8231961/webrev.00/

In C1, flattened array loads are connected to a buffer allocation (NewValueTypeInstance) to write
the result into. If there are multiple loads from the same array element, local/global value
numbering should not remove them because they are initializing different buffers. The fix is to
simply make the hashing code aware of the _vt field of the LoadIndexed instruction.

Thanks,
Tobias


More information about the valhalla-dev mailing list