[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