[lworld] Integrated: 8273594: [lworld] JITs need to properly handle static inline type field with unloaded type

Tobias Hartmann thartmann at openjdk.java.net
Fri Oct 15 06:02:13 UTC 2021


On Fri, 10 Sep 2021 11:19:48 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:

> Both C1 and C2 do not properly handle loads from static inline type fields with an unloaded type. For C1, the fix is to simply remove two asserts that are too strong. For C2, we need to trap during typeflow analysis.
> 
> I've added a corresponding test to `TestUnloadedInlineTypeField.java` and noticed that the new IR Test Framework often triggers class loading while the test was carefully designed to avoid that. As a workaround, I slightly modified the framework and run the test with `-DIgnoreCompilerControls=true`. I filed [JDK-8273591](https://bugs.openjdk.java.net/browse/JDK-8273591) to fix this upstream.
> 
> I also fixed a `-XX:-XX:+PatchALot` typo in the test that went unnoticed because `-XX:+IgnoreUnrecognizedVMOptions` is set.
> 
> Thanks,
> Tobias

This pull request has now been integrated.

Changeset: 902a1577
Author:    Tobias Hartmann <thartmann at openjdk.org>
URL:       https://git.openjdk.java.net/valhalla/commit/902a15773118364db4ea38c2cc9c5f93f49c1394
Stats:     264 lines in 7 files changed: 234 ins; 9 del; 21 mod

8273594: [lworld] JITs need to properly handle static inline type field with unloaded type

Reviewed-by: fparain

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

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



More information about the valhalla-dev mailing list