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

Tobias Hartmann thartmann at openjdk.java.net
Wed Jul 28 09:49:58 UTC 2021


`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

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

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

Changes: https://git.openjdk.java.net/valhalla/pull/504/files
 Webrev: https://webrevs.openjdk.java.net/?repo=valhalla&pr=504&range=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8271380
  Stats: 128 lines in 9 files changed: 85 ins; 0 del; 43 mod
  Patch: https://git.openjdk.java.net/valhalla/pull/504.diff
  Fetch: git fetch https://git.openjdk.java.net/valhalla pull/504/head:pull/504

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



More information about the valhalla-dev mailing list