[lworld] RFR: 8359370: [lworld] allow instance fields of identity classes to be readable in the prologue phase [v7]
Vicente Romero
vromero at openjdk.org
Mon Aug 25 12:58:06 UTC 2025
On Fri, 22 Aug 2025 16:50:53 GMT, Vicente Romero <vromero at openjdk.org> wrote:
>> Vicente Romero has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 17 commits:
>>
>> - fixing bugs: new test cases brought in with latest merge are not accepted
>> - merge with lworld
>> - refactorings
>> - another bug fix
>> - bug fix
>> - merge with lworld
>> - renaming methods in LocalProxyVarGen
>> - more changes
>> - refactoring
>> - test changes
>> - ... and 7 more: https://git.openjdk.org/valhalla/compare/62e90b5d...0eb81681
>
> src/jdk.compiler/share/classes/com/sun/tools/javac/comp/LocalProxyVarsGen.java line 137:
>
>> 135: .filter(t -> t.hasTag(VARDEF))
>> 136: .map(t -> (JCVariableDecl)t)
>> 137: .filter(vd -> (vd.sym.isStrict() || vd.sym.isFinal()) && !vd.sym.isStatic())
>
> semantic change
doing some changes here, I realized there is a bug in the PR, working on it now
-------------
PR Review Comment: https://git.openjdk.org/valhalla/pull/1523#discussion_r2298029304
More information about the valhalla-dev
mailing list