[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