-
-
Notifications
You must be signed in to change notification settings - Fork 298
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[DYNAREC] Implement perf map #2212
Conversation
Thank you! While it's cool, in my opinion, I don't think showing the x86 instruction name is of much use to us. Perf maps are mainly used for perfing guest programs to make it easier to find hot code, so it will be useful for users to find hot code in Java or Python programs with this implemented in JVM or Python VM, but it is not helpful for JVM developers. But in any case, I will wait for @ptitSeb to review this. |
I would prefer the perf-map to represent the name of the x64 function (if any), or the x64 address for anonymous map (that could be later named after the file name map when this will be tracked properly), and then use a simple hex offset for each instruction instead of the opcode name that is kind of anonymous at tis level. If you think you can easily do the change, I can wait. I you prefer this pr to be merge and look at a "2" level with function name, I can merge this quickly (but indeed, I don't see this function really usefull in it's current state) |
Yup, JVM use the function alike (method or jvmci) name: https://github.com/openjdk/jdk/blob/master/src/hotspot/share/code/codeCache.cpp#L1833 Thanks, |
Use function name as the Symbol:
Please review my patch again. Thanks, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The PR looks good to me now.
LGTM too. |
Thanks 🍻 |
Hi,
To find out DynaRec jit "hot" code:
scimark: https://math.nist.gov/scimark2/download_c.html
Before no perf map no Symbol:
After generated perf map just use x86 INST NAME as the Symbol name:
Please review my patch and give some suggestion.
Thanks,
Leslie Zhai