[lworld] RFR: 8244231: [lworld] Add support for ref-default and val-default inline classes.
Srikanth Adayapalam
sadayapalam at openjdk.java.net
Mon Jul 19 06:21:05 UTC 2021
On Fri, 16 Jul 2021 08:05:02 GMT, Srikanth Adayapalam <sadayapalam at openjdk.org> wrote:
> Code changes and tests
src/jdk.compiler/share/classes/com/sun/tools/javac/code/Types.java line 1216:
> 1214: && (t.tsym != s.tsym ||
> 1215: (t.isReferenceProjection() == s.isReferenceProjection() && t.isValueProjection() == s.isValueProjection()))
> 1216: // Check type variable containment
This check should really be comparing that t.flavor == s.flavor. But this is causing some issues at the moment and will be followed up in JDK-8270882
src/jdk.compiler/share/classes/com/sun/tools/javac/code/Types.java line 1463:
> 1461: && t.isReferenceProjection() == s.isReferenceProjection()
> 1462: && t.isValueProjection() == s.isValueProjection()
> 1463: && visit(getEnclosingType(t), getEnclosingType(s))
Likewise this should be checking that t.flavor == s.flavor. Will be followed up in JDK-8270882
-------------
PR: https://git.openjdk.java.net/valhalla/pull/482
More information about the valhalla-dev
mailing list