-
Notifications
You must be signed in to change notification settings - Fork 721
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 native method stubs for jdk.jfr.internal.jvm #20053
base: master
Are you sure you want to change the base?
Conversation
@tajila FYI |
9d000ae
to
77eefea
Compare
@@ -21,4 +21,69 @@ SPDX-License-Identifier: EPL-2.0 OR Apache-2.0 OR GPL-2.0-only WITH Classpath-ex | |||
--> | |||
<exports group="jfr"> | |||
<export name="Java_com_ibm_oti_vm_VM_triggerExecutionSample" /> | |||
<export name="Java_jdk_jfr_internal_JVM_registerNatives" /> |
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.
youll also need a objects.xml for UMA build system
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.
then youll need to update runtime/jcl/module.xml to add both the objects and the exports file
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.
Sorry for the late reply, I've done the changes
77eefea
to
ba26fd8
Compare
SPDX-License-Identifier: EPL-2.0 OR Apache-2.0 OR GPL-2.0-only WITH Classpath-exception-2.0 OR GPL-2.0-only WITH OpenJDK-assembly-exception-1.0 | ||
--> | ||
<objects group="jfr"> |
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.
should also have
<object name="Java_jdk_jfr_internal_JVM" >
<include-if condition="spec.flags.opt_jfr" />
</object>
@@ -65,6 +65,7 @@ SPDX-License-Identifier: EPL-2.0 OR Apache-2.0 OR GPL-2.0-only WITH Classpath-ex | |||
<xi:include href="uma/se9_objects.xml"></xi:include> | |||
<xi:include href="uma/se16_objects.xml"></xi:include> | |||
<xi:include href="uma/se19_objects.xml"></xi:include> | |||
<xi:include href="uma/jfr_objects.xml"></xi:include> |
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.
you need an entry in the objects list below as well
<group name="jfr">
<include-if condition="spec.flags.opt_jfr"/>
</group>
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.
Try building this in the internal farm with vmfarm on java8. Also try building it with openj9 with --without-cmake
config flag
2f123ed
to
95d5b08
Compare
Signed-off-by: Gengchen Tuo <[email protected]>
95d5b08
to
5fb7ab1
Compare
No description provided.