[lworld] RFR: 8273594: [lworld] JITs need to properly handle static inline type field with unloaded type [v5]
Tobias Hartmann
thartmann at openjdk.java.net
Thu Oct 14 11:55:45 UTC 2021
> 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
Tobias Hartmann has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains ten additional commits since the last revision:
- Test fix
- Merge branch 'lworld' into JDK-8273594
- Fix
- Remove -Xint from CircularityTest
- Merge branch 'lworld' into JDK-8273594
- Check for uninitialized class
- Deoptimization does no longer support symbolic names for the class index, use a simple trap instead
- Don't modify do_Deoptimize
- 8273594: [lworld] JITs need to properly handle static inline type field with unloaded type
-------------
Changes:
- all: https://git.openjdk.java.net/valhalla/pull/551/files
- new: https://git.openjdk.java.net/valhalla/pull/551/files/35a1c2a7..4f2a467a
Webrevs:
- full: https://webrevs.openjdk.java.net/?repo=valhalla&pr=551&range=04
- incr: https://webrevs.openjdk.java.net/?repo=valhalla&pr=551&range=03-04
Stats: 62387 lines in 2034 files changed: 40281 ins; 11984 del; 10122 mod
Patch: https://git.openjdk.java.net/valhalla/pull/551.diff
Fetch: git fetch https://git.openjdk.java.net/valhalla pull/551/head:pull/551
PR: https://git.openjdk.java.net/valhalla/pull/551
More information about the valhalla-dev
mailing list