[lworld] RFR: 8253592: [lworld] C2: Additional fixes for handling empty inline types [v2]
Tobias Hartmann
thartmann at openjdk.java.net
Thu Sep 24 12:48:14 UTC 2020
> While working on JDK-8253416, I found more issues with handling of empty inline types in C2:
> - Compile::build_start_state assumes that all inline type arguments are scalarized if has_scalarized_args() is true but
> one of the arguments could be non-scalarized (for example, if it's an empty inline type).
> - Loads/stores from/to inline type fields should be removed. Refactoring the related code also fixes an issue where alias
> analysis gets confused when looking at flattened fields of empty inline types.
>
> Unrelated fixes:
> - Handling of unexpected inline/non-inline klasses in _new/_defaultvalue.
> - Missing @DontCompile statements in some tests
Tobias Hartmann has updated the pull request incrementally with one additional commit since the last revision:
Disabling some tests until JDK-8253416 is fixed
-------------
Changes:
- all: https://git.openjdk.java.net/valhalla/pull/197/files
- new: https://git.openjdk.java.net/valhalla/pull/197/files/ea4d32d4..4c32cea6
Webrevs:
- full: https://webrevs.openjdk.java.net/?repo=valhalla&pr=197&range=01
- incr: https://webrevs.openjdk.java.net/?repo=valhalla&pr=197&range=00-01
Stats: 3 lines in 1 file changed: 3 ins; 0 del; 0 mod
Patch: https://git.openjdk.java.net/valhalla/pull/197.diff
Fetch: git fetch https://git.openjdk.java.net/valhalla pull/197/head:pull/197
PR: https://git.openjdk.java.net/valhalla/pull/197
More information about the valhalla-dev
mailing list