[lworld] Integrated: 8262128: [lworld] C1's ValueNumbering optimization does not correctly handle delayed accesses

Tobias Hartmann thartmann at openjdk.java.net
Tue Feb 23 06:58:53 UTC 2021


On Mon, 22 Feb 2021 14:06:35 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:

> C1's ValueNumbering optimization replaces an array+field load `array[0].intArray` by another non-equivalent array+field load `array[0].floatArray` because it does not handle delayed accesses. The hash value of `LoadIndexed` needs to take delayed accesses into account.
> 
> Best regards,
> Tobias

This pull request has now been integrated.

Changeset: 1c9a7a9a
Author:    Tobias Hartmann <thartmann at openjdk.org>
URL:       https://git.openjdk.java.net/valhalla/commit/1c9a7a9a
Stats:     34 lines in 3 files changed: 25 ins; 2 del; 7 mod

8262128: [lworld] C1's ValueNumbering optimization does not correctly handle delayed accesses

Reviewed-by: fparain

-------------

PR: https://git.openjdk.java.net/valhalla/pull/345


More information about the valhalla-dev mailing list