[lworld] Integrated: 8253592: [lworld] C2: Additional fixes for handling empty inline types

Tobias Hartmann thartmann at openjdk.java.net
Thu Sep 24 14:07:01 UTC 2020


On Thu, 24 Sep 2020 11:26:13 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:

> 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

This pull request has now been integrated.

Changeset: 33bdce23
Author:    Tobias Hartmann <thartmann at openjdk.org>
URL:       https://git.openjdk.java.net/valhalla/commit/33bdce23
Stats:     209 lines in 10 files changed: 122 ins; 50 del; 37 mod

8253592: [lworld] C2: Additional fixes for handling empty inline types

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

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


More information about the valhalla-dev mailing list