[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