RFR: 8366461: Remove obsolete method handle invoke logic [v3]
Dean Long
dlong at openjdk.org
Thu Sep 4 00:31:42 UTC 2025
On Wed, 3 Sep 2025 07:12:20 GMT, Manuel Hässig <mhaessig at openjdk.org> wrote:
>> Dean Long has updated the pull request incrementally with three additional commits since the last revision:
>>
>> - revert whitespace change
>> - undo debug changes
>> - cleanup
>
> src/jdk.hotspot.agent/share/classes/sun/jvm/hotspot/runtime/aarch64/AARCH64Frame.java line 372:
>
>> 370: // DEBUG_ONLY(verifyDeoptriginalPc(senderNm, raw_unextendedSp));
>> 371: }
>> 372: }
>
> `<arch>Frame.java adjustUnextendedSP()` do not seem to do anything? Perhaps these could be cleaned up as well?
Yes, it's tempting to want to clean these up, but I noticed that SA code really tries to mirror the C++ code, so I'm inclined to leave it. Is there a Serviceability expert that would like to see this code cleaned up further? @plummercj , what do you think?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/27059#discussion_r2320526360
More information about the hotspot-dev
mailing list