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

[Coleen Phillimore]

On Fri, 13 Jun 2025 13:58:37 GMT, Matias Saavedra Silva <matsaave at openjdk.org> wrote:

>> Need to write a test.
>
> 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.

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

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