[lworld] RFR: 8359377: [lworld] Fix cp indexes in larval stackmaps in redefinition

Coleen Phillimore coleenp at openjdk.org
Mon Jun 16 11:55:37 UTC 2025


On Fri, 13 Jun 2025 15:56:45 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> src/hotspot/share/prims/jvmtiRedefineClasses.cpp line 3318:
>> 
>>> 3316: 
>>> 3317:     // reserved for future use
>>> 3318:     else if (frame_type >= 128 && frame_type <= 245) {
>> 
>> This is beyond the scope of your fix but I think we should have an enum for the frame types here just like in stackMapTable.cpp.
>
> I agree.

I'll have a look at fixing that in mainline first.

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

PR Review Comment: https://git.openjdk.org/valhalla/pull/1487#discussion_r2149778440


More information about the valhalla-dev mailing list