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

[GR-61448] Include Compilation ID in truffle compilation logs. #10549

Merged
merged 3 commits into from
Jan 24, 2025

Conversation

graalvmbot
Copy link
Collaborator

No description provided.

JohnTortugo and others added 3 commits January 13, 2025 17:54
…into jchaloup/GR-61448/CompIdInTruffleCompLogs
… id logging work for libgraal and isolated truffle compiler on SVM; add the compilation id also to the truffle compilation JFR logs.
@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Jan 23, 2025
@graalvmbot graalvmbot closed this Jan 24, 2025
@graalvmbot graalvmbot deleted the jchaloup/GR-61448/CompIdInTruffleCompLogs branch January 24, 2025 00:51
@graalvmbot graalvmbot merged commit d3c4e02 into master Jan 24, 2025
41 checks passed

import com.oracle.truffle.compiler.hotspot.libgraal.TruffleToLibGraal;

final class TruffleToLibGraalCalls2 {
Copy link
Contributor

Choose a reason for hiding this comment

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

@jchalou - Out of curiosity: why do we need a separate file for this method?

Copy link
Member

Choose a reason for hiding this comment

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

This is because Truffle has to be compatible also with libgraal that does not have the getCompilationId method entry point. LibGraalTruffleCompilationSupport registers the native methods is a way that either registers all the native methods in a class or none at all, and so the ones that are not required to be there have to be in their separate classes (for now, we do plan to introduce some more sophisticated methodology in the future).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OCA Verified All contributors have signed the Oracle Contributor Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants