[lworld] RFR: 8376843: [lworld] add more regression tests for local variable proxies

Vicente Romero vromero at openjdk.org
Sun Feb 1 01:14:24 UTC 2026


On Sun, 1 Feb 2026 01:06:22 GMT, Vicente Romero <vromero at openjdk.org> wrote:

> adding some additional regression tests for local proxy variables
> 
> TIA

src/jdk.compiler/share/classes/com/sun/tools/javac/comp/Attr.java line 1548:

> 1546:                                 if (mode == PrologueVisitorMode.THIS_CONSTRUCTOR) {
> 1547:                                     reportPrologueError(tree, sym);
> 1548:                                 } else if (mode == PrologueVisitorMode.SUPER_CONSTRUCTOR && isInstanceField(tree)) {

not necessary with the current implementation but, not bad to be extra sure, future proof

test/langtools/tools/javac/valhalla/value-objects/LocalProxyVariablesTests.java line 86:

> 84:         doTest(
> 85:                 """
> 86:                 value class Test1 {

even though the compiler used is supposed to be reusable, using the same name for all classes produces an error, would be nice to fix this issue so that we can have a 100% reusable compiler

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

PR Review Comment: https://git.openjdk.org/valhalla/pull/2008#discussion_r2750324183
PR Review Comment: https://git.openjdk.org/valhalla/pull/2008#discussion_r2750327255


More information about the valhalla-dev mailing list