[lworld] Integrated: 8281454: [lworld] Assert in EA due to oop access to flat array

Tobias Hartmann thartmann at openjdk.java.net
Wed Feb 9 11:39:37 UTC 2022


On Wed, 9 Feb 2022 11:05:10 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:

> Similar to [JDK-8250951](https://bugs.openjdk.java.net/browse/JDK-8250951), we hit an assert in EA because an oop load is used to access a flat array. The load is from an `Arrays.copyOf` and flat array checks in `LibraryCallKit::inline_array_copyOf` will prevent that load from ever being executed but the dead subgraph is not removed because we run with `-XX:+StressReflectiveCode`.
> 
> I fixed this by using the `FlatArrayCheckNode` for checks in the intrinsics as well. This ensures that the graph is folded even with  `-XX:+StressReflectiveCode` and enables more aggressive optimizations. The patch also includes some cleanups and removal of (now) dead code.
> 
> Thanks,
> Tobias

This pull request has now been integrated.

Changeset: 5a0f3336
Author:    Tobias Hartmann <thartmann at openjdk.org>
URL:       https://git.openjdk.java.net/valhalla/commit/5a0f3336c0e51ea8e4942f7b70a40f37ded0c2f2
Stats:     83 lines in 12 files changed: 10 ins; 22 del; 51 mod

8281454: [lworld] Assert in EA due to oop access to flat array

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

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



More information about the valhalla-dev mailing list