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

Adapt cross package interface test to correctly handle Java 8 #18398

Merged
merged 1 commit into from Nov 7, 2023

Conversation

ThanHenderson
Copy link
Contributor

@ThanHenderson ThanHenderson commented Nov 3, 2023

This patch fixes #14987. OJDK MHs use OJDK's version of findSpecial. Before JEP 274, which was implemented for Java 9, findSpecial did not support interface method access. This patch handles the IllegalAccessException that is thrown for OpenJ9 with OJDK MHs enabled.

Closes: #14987
Signed-off-by: Nathan Henderson nathan.henderson@ibm.com

@babsingh
Copy link
Contributor

babsingh commented Nov 6, 2023

In the commit message:

  • This patch fixes both #14987: is the second reference missing (both)?
  • Java8 -> Java 8
  • Java9 -> Java 9

@ThanHenderson
Copy link
Contributor Author

This patch fixes both #14987: is the second reference missing (both)?

Simply a commit message template error. Good catch.

This patch fixes eclipse-openj9#14987. OJDK MHs use OJDK's
version of findSpecial. Before JEP 274, which was implemented for
Java 9, findSpecial did not support interface method access. This
patch handles the IllegalAccessException that is thrown for OpenJ9
with OJDK MHs enabled.

Closes: eclipse-openj9#14987
Signed-off-by: Nathan Henderson <nathan.henderson@ibm.com>
@ThanHenderson ThanHenderson changed the title Adapt cross package interface test to correctly handle Java8 Adapt cross package interface test to correctly handle Java 8 Nov 6, 2023
@babsingh
Copy link
Contributor

babsingh commented Nov 6, 2023

jenkins test extended.functional zlinuxojdk292 jdk8,jdk11

@babsingh
Copy link
Contributor

babsingh commented Nov 6, 2023

jenkins test extended.functional plinux jdk8,jdk11

@babsingh babsingh merged commit 3ba8567 into eclipse-openj9:master Nov 7, 2023
9 of 11 checks passed
@ThanHenderson ThanHenderson added jdk8 comp:vm project:MH Used to track Method Handles related work labels Jan 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comp:vm jdk8 project:MH Used to track Method Handles related work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[JDK8 OJDK-MH] IAE from test_FindSpecial_Default_CrossPackage_Interface
2 participants