[lworld] [Rev 01] RFR: 8243625: [lworld] Co-evolve hotspot-compiler tests along with JDK-8237072
sadayapalam at openjdk.java.net
Fri May 1 15:10:35 UTC 2020
> May I request you to review these changes to hotspot-compiler 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.
> 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.
Srikanth Adayapalam has updated the pull request incrementally with one additional commit since the last revision:
Include fixes from Tobias for compilation failures.
- all: https://git.openjdk.java.net/valhalla/pull/30/files
- new: https://git.openjdk.java.net/valhalla/pull/30/files/035020ab..9d1f824a
- full: https://webrevs.openjdk.java.net/valhalla/30/webrev.01
- incr: https://webrevs.openjdk.java.net/valhalla/30/webrev.00-01
Stats: 22 lines in 3 files changed: 9 ins; 0 del; 13 mod
Fetch: git fetch https://git.openjdk.java.net/valhalla pull/30/head:pull/30
More information about the valhalla-dev