[lworld] Integrated: 8271380: [lworld] Intrinsify j.l.Class::asPrimaryType/asValueType

Tobias Hartmann thartmann at openjdk.java.net
Wed Jul 28 11:49:48 UTC 2021


On Wed, 28 Jul 2021 09:43:35 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:

> `TestIntrinsics.java` fails after [JDK-8269956](https://bugs.openjdk.java.net/browse/JDK-8269956) which replace some usages of `MyValue.class` by `MyValue.class.asValueType()`. The problem is that `asValueType()` is not always inlined by C2 and therefore IR verification fails.
> 
> This patch intrinsifies the methods similar to what we did in old L/Q world (https://github.com/openjdk/valhalla/commit/e895128f). I've also modified the test to consistently use `asPrimaryType`/`asValueType`.
> 
> Thanks,
> Tobias

This pull request has now been integrated.

Changeset: bebb40f5
Author:    Tobias Hartmann <thartmann at openjdk.org>
URL:       https://git.openjdk.java.net/valhalla/commit/bebb40f5dc00b40f1fcdb68f9cebbd7bb597095d
Stats:     128 lines in 9 files changed: 85 ins; 0 del; 43 mod

8271380: [lworld] Intrinsify j.l.Class::asPrimaryType/asValueType

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

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


More information about the valhalla-dev mailing list