[lworld] Integrated: 8280382: [lworld] JCWithField AST node support should evolve to work with value classes

Aggelos Biboudis duke at openjdk.java.net
Fri Jan 28 15:46:37 UTC 2022


On Thu, 27 Jan 2022 19:24:47 GMT, Aggelos Biboudis <duke at openjdk.java.net> wrote:

> This PR ports basic `withField` tests from lworld-values to value-objects and extending some areas of code to support `withField` within value classes.
> 
> Nitpicking and a note: one of the error messages is now `compiler.err.primitive.or.value.class.instance.field.expected.here`. Would you propose to disambiguate further depending whether we are inside a `primitive` or a `value` class?

This pull request has now been integrated.

Changeset: d41022dc
Author:    Angelos Bimpoudis <angelos.bimpoudis at oracle.com>
Committer: Srikanth Adayapalam <sadayapalam at openjdk.org>
URL:       https://git.openjdk.java.net/valhalla/commit/d41022dc679bc275dd57ef8f2a1c9bf817d559d8
Stats:     597 lines in 18 files changed: 585 ins; 0 del; 12 mod

8280382: [lworld] JCWithField AST node support should evolve to work with value classes

Reviewed-by: sadayapalam

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

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



More information about the valhalla-dev mailing list