[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