[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