[lworld] RFR: 8243623: [lworld] Syntax and other mechanical changes in langtools tests for JDK-8237072
Srikanth Adayapalam
sadayapalam at openjdk.java.net
Mon Apr 27 04:47:26 UTC 2020
Jim,
May I request you to review these changes to langtools tests that anticipate and
align with JDK-8237072 ? These changes are mechanical in nature and I decided it
is best to separate them from the RFR for JDK-8237072 itself to help create sharper
focus on the important pieces there. (Non-mechanical meaningful test changes with
be included in that RFR itself)
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:
- 8243623: [lworld] Syntax and other mechanical changes in langtools tests for JDK-8237072
Changes: https://git.openjdk.java.net/valhalla/pull/29/files
Webrev: https://webrevs.openjdk.java.net/valhalla/29/webrev.00
Issue: https://bugs.openjdk.java.net/browse/JDK-8243623
Stats: 90 lines in 26 files changed: 0 ins; 0 del; 90 mod
Patch: https://git.openjdk.java.net/valhalla/pull/29.diff
Fetch: git fetch https://git.openjdk.java.net/valhalla pull/29/head:pull/29
PR: https://git.openjdk.java.net/valhalla/pull/29
More information about the valhalla-dev
mailing list