On Thu, 6 Aug 2026 23:57:32 GMT, Chen Liang <[email protected]> wrote:
>> [JDK-8376522](https://bugs.openjdk.org/browse/JDK-8376522) indicates that >> the thisUninit should be applied even when uninitializedThis is on the >> stack. Previous implementations failed to do this, so this patch properly >> enforces this spec change. >> >> Now that both the stack and locals must be considered when applying >> flagThisUninit, the origin of the flag needs to be preserved, so the >> StackMapReader now tracks if the flag originated from locals or the stack. >> Since the stack is cleared between frames, only an uninitializedThis from >> locals is preserved. >> >> In addition to VM changes, the ClassFile API had to be updated as it only >> considered stack frame locals. The test case offered by @liach in the JBS >> issue has been adapted to a JASM test to verify the change. Verified with >> tier 1-5 tests. >> >> --------- >> - [x] I confirm that I make this contribution in accordance with the >> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai). > > You can rollback the jdk/internal/classfile changes: They are fixed in > https://github.com/openjdk/jdk/pull/32224 Thanks for the reviews and suggestions @liach @DanHeidinga and @fparain! ------------- PR Comment: https://git.openjdk.org/jdk/pull/32242#issuecomment-5241860511
