On Fri, 7 Aug 2026 19:03:44 GMT, Dan Heidinga <[email protected]> wrote:

>> Matias Saavedra Silva has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   Cleanup and test case improvement
>
> src/hotspot/share/classfile/stackMapTable.cpp line 234:
> 
>> 232: 
>> 233:       // An uninitializedThis in the locals array can be preserved 
>> between
>> 234:       // frames while uninitializedThis in the stack cannot as the 
>> stack is cleared.
> 
> This isn't accurate.  Most frames preserve the locals but not all - ie: a 
> `chop_frame` is an empty stack that removes the last `k` locals.

I figured the word "can" would imply that but I can rephrase and add more 
detail here.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/32242#discussion_r3738445736

Reply via email to