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. Actually `printmdo` is not so needed for troubleshooting POV, but it is useful for learning HotSpot and JIT compiler behavior IMHO (I gave a talk about MDO in relation to JVMCI in past ( @ Java Day Tokyo 2017)). Replay data might be much better as you wrote in CSR, but it is difficult to extract MDO from that, and also it is difficult to check realtime data (e.g. num of branches). I do not have big objections for removing `printmdo`, but I think it is better if we can keep it for learning HotSpot if possible. I'm not familiar with Valhalla, but I want to know whether it needs big changes to support `printmdo`. ------------- PR Comment: https://git.openjdk.org/jdk/pull/29693#issuecomment-3900992587
