On Mon, 10 Aug 2026 20:42:23 GMT, Chen Liang <[email protected]> wrote:

>> We should allow unset fields when `uninitializedThis` is in the stack, in 
>> addition to only checking locals, as a result of 
>> https://bugs.openjdk.org/browse/JDK-8375481.
>> 
>> Noticed that hotspot's verifier is not up-to-date with this spec; this patch 
>> therefore only fixes classfile API and javap and leaves classfile API's 
>> verifier and hotspot in place.
>> 
>> ---------
>> - [x] I confirm that I make this contribution in accordance with the 
>> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai).
>
> Chen Liang has updated the pull request with a new target base due to a merge 
> or a rebase. The incremental webrev excludes the unrelated changes brought in 
> by the merge/rebase. The pull request contains three additional commits since 
> the last revision:
> 
>  - Add 8389843
>  - Merge branch 'master' of https://github.com/openjdk/jdk into 
> fix/cf-unset-restriction
>  - unset field allowed for uninitThis on stack

I’m finding this PR hard to review in its current form. 
It appears to combine several related changes into a complex refactoring, 
however their motivation and relationship are not described. 
Title and description seem to cover only part of the story.

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

PR Comment: https://git.openjdk.org/jdk/pull/32224#issuecomment-5252307376

Reply via email to