Skip to content
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

Add support for jcmd Compiler.perfmap #298

Merged
merged 1 commit into from
May 31, 2024
Merged

Conversation

theothornhill
Copy link
Contributor

@theothornhill theothornhill commented Apr 30, 2024

Even though the perf "spec" for jit languages define the perf map syntax, the jcmd Compiler.perfmap insists on prepending 0x to the addresses.

In addition, there's now a way for the jvm to generate the perfmap automatically on exit, making magic-trace run -multi-thread -snapshot-size 1M -- ./demo/java return something useful without messing around with perfmap generation. Add these options to the ./demo/java file

@theothornhill
Copy link
Contributor Author

Ping! 😊

Copy link
Member

@Xyene Xyene left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks for the contribution!

@theothornhill
Copy link
Contributor Author

Thanks - I'll look at the formatting failure. Code was run using the examples from the readme. I'll check if that either needs amending or if I missed something.

Even though the perf "spec" for jit languages defines the perf map syntax, the
jcmd Compiler.perfmap insists on prepending 0x to the addresses.

In addition, there's now a way for the jvm to generate the perfmap automatically
on exit, making `magic-trace run -multi-thread -snapshot-size 1M -- ./demo/java`
return something useful without messing around with perfmap generation. Add
these options to the ./demo/java file

Signed-off-by: Theodor Thornhill <theo@thornhill.no>
@theothornhill
Copy link
Contributor Author

I updated the .ocamlformat to 0.26.2. Do you want me to also update the relevant parts of the documentation?

@Xyene
Copy link
Member

Xyene commented May 31, 2024

I've updated the docs, thanks!

@Xyene Xyene merged commit a8d3a3c into janestreet:master May 31, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants