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

[FFI] Remove all code & test suites tagged with J9VM_OPT_PANAMA #19862

Merged

Conversation

ChengJin01
Copy link

@ChengJin01 ChengJin01 commented Jul 17, 2024

The changes remove the stale code & tests tagged with J9VM_OPT_PANAMA
in JDK16 given all FFI related functionalities were fully implemented
in which case the code & test surrounded by or related to
J9VM_OPT_PANAMA are no longer required for any reference.

Signed-off-by: ChengJin01 [email protected]

Fixes: #10377.

@ChengJin01
Copy link
Author

The PR has been verified in personal builds (internal & open) on all supported platforms.

Reviewer: @keithc-ca
FYI: @tajila, @pshipton

@ChengJin01 ChengJin01 requested a review from keithc-ca July 17, 2024 04:38
@ChengJin01 ChengJin01 added comp:vm project:panama Used to track Project Panama related work labels Jul 17, 2024
@keithc-ca
Copy link
Contributor

Jenkins test sanity amac jdk11,jdk23

runtime/vm/MHInterpreter.inc Outdated Show resolved Hide resolved
@keithc-ca
Copy link
Contributor

For the record, testing passed at https://openj9-jenkins.osuosl.org/job/PullRequest-OpenJ9/5841.

The changes remove the stale code & tests tagged with J9VM_OPT_PANAMA
in JDK16 given all FFI related functionalities were fully implemented
in which case the code & test surrounded by or related to 
J9VM_OPT_PANAMA are no longer required for any reference.

Signed-off-by: ChengJin01 <[email protected]>
@ChengJin01 ChengJin01 force-pushed the ffi_remove_stale_panama_code branch from a58f8c6 to 950eec7 Compare July 17, 2024 16:11
@keithc-ca keithc-ca merged commit bb41a03 into eclipse-openj9:master Jul 17, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comp:vm project:panama Used to track Project Panama related work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove PANAMA code
2 participants