RFR: 8377797: Remove SA support for MethodData and the printmdo command [v3]

[Chris Plummer]

On Fri, 13 Feb 2026 18:24:57 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> This change removes MethodData code copied into SA that's out of date for the valhalla repo.
>> 
>> Tested with tier1-3.
>
> Coleen Phillimore has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Remove the doc.

src/jdk.hotspot.agent/share/classes/sun/jvm/hotspot/CommandProcessor.java line 863:

> 861:         },
> 862:         new Command("printall", "printall", false) {
> 863:             // Print every MDO in the heap or the one referenced by expression.

I just noticed this comment for `printall` and it doesn't seem right. It mentions printing MDOs, but that is what you are removing. The implementation seems to print each Method. Maybe that normally includes MethodData, but I don't think so since I don't see any signs of it getting the MethodData and calling mdo.printDataOn() like `printmdo` does. Maybe all we need here is a minor comment update to replace "MDO" with "Method".

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

PR Review Comment: https://git.openjdk.org/jdk/pull/29693#discussion_r2818696370