[lworld] RFR: 8243627: [lworld] Co-evolve jdk-valhalla tests along with JDK-8237072

Srikanth Adayapalam sadayapalam at openjdk.java.net
Mon Apr 27 04:23:08 UTC 2020


Mandy,

May I request you to review these changes to jdk-valhalla tests ?

JDK-8237072 adds supports for the new syntax notation of V.ref and V.val to
refer to the reference projection of a value type V and its value projection.
The old syntax of V? is withdrawn. This change also has class file implications
where the descriptor/signature encodings will now start mentioning $ref in the
class pool entries. Also every inline type results in two class files now
one for each projection - with the reference projection class being the superclass
of the inline class.

I'll push them after your review and *after* JDK-8237072 itself is pushed.
Thanks in advance.

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

Commit messages:
 - 8243627: [lworld] Co-evolve jdk-valhalla tests along with JDK-8237072

Changes: https://git.openjdk.java.net/valhalla/pull/27/files
 Webrev: https://webrevs.openjdk.java.net/valhalla/27/webrev.00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8243627
  Stats: 52 lines in 11 files changed: 1 ins; 0 del; 51 mod
  Patch: https://git.openjdk.java.net/valhalla/pull/27.diff
  Fetch: git fetch https://git.openjdk.java.net/valhalla pull/27/head:pull/27

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


More information about the valhalla-dev mailing list