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

Tobias Hartmann thartmann at openjdk.java.net
Mon Feb 22 14:11:24 UTC 2021


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

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

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

Changes: https://git.openjdk.java.net/valhalla/pull/345/files
 Webrev: https://webrevs.openjdk.java.net/?repo=valhalla&pr=345&range=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8262128
  Stats: 44 lines in 4 files changed: 35 ins; 2 del; 7 mod
  Patch: https://git.openjdk.java.net/valhalla/pull/345.diff
  Fetch: git fetch https://git.openjdk.java.net/valhalla pull/345/head:pull/345

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


More information about the valhalla-dev mailing list