On Fri, 13 Feb 2026 18:24:57 GMT, Coleen Phillimore <[email protected]> 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.

I mean `printmdo` might be useful for understand VM behavior and method usage 
in HotSpot POV.
For example, the user attaches CLHSDB and run `printmdo` to get MDO as 
baseline, then run again 10 minutes later. The user can know how the method was 
run and HotSpot collect statistics during 10 mins.

If I understand correctly, `PrintMethodData` and `CompileCommand=print` would 
show statistics when VM is exitting ( @ `before_exit()` ). We cannot use them 
in this use case.

Anyway, I almost agree to remove `printmdo` because it is not useful for 
troubleshooting IMHO. If the user want to get MDO for learning, they can attach 
native debugger...

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

PR Comment: https://git.openjdk.org/jdk/pull/29693#issuecomment-3917734072

Reply via email to