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

Added configuration option to support MicroJIT builds of OpenJ9 #413

Open
wants to merge 5 commits into
base: openj9
Choose a base branch
from

Conversation

Scott-Young-6746
Copy link

@Scott-Young-6746 Scott-Young-6746 commented May 15, 2020

This PR adds support for an --enable-microjit flag during configuration to support an OpenJ9 build with the MicroJIT project code. This PR should not prevent an unmodified OpenJ9 from compiling nor should enabling the --enable-microjit flag with an unmodified OpenJ9 impact the build output.

See also eclipse-openj9/openj9#9578

Signed-off-by: Scott Young scott.young@unb.ca

closed/OpenJ9.gmk Outdated Show resolved Hide resolved
@keithc-ca
Copy link
Member

Unless there are limitations in eclipse-openj9/openj9#9578 that exclude support for newer Java versions, this should be repeated in jdk11, jdk14 and jdknext.

@Scott-Young-6746
Copy link
Author

Unless there are limitations in eclipse/openj9#9578 that exclude support for newer Java versions, this should be repeated in jdk11, jdk14 and jdknext.

At this time we haven't tested MicroJIT on any other JDKs than jdk8.

However, I can't think of any reason it wouldn't work.

@pshipton
Copy link
Member

Java 11 adds new bytecode for eclipse-openj9/openj9#954

Scott-Young-6746 and others added 5 commits May 13, 2022 13:34
Signed-off-by: Scott Young <scott.young@unb.ca>
Signed-off-by: Scott Young <scott.young@unb.ca>
Signed-off-by: Scott Young <scott.young@unb.ca>
Signed-off-by: Harpreet Kaur <harpreet.kaur5@ibm.com>
Signed-off-by: Harpreet Kaur <hkaur3@unb.ca>
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.

4 participants