Skip to content

Disable latest JDK22 APIs for Valhalla - #18327

Merged
keithc-ca merged 1 commit into
eclipse-openj9:masterfrom
JasonFengJ9:fixvalhalla
Oct 23, 2023
Merged

Disable latest JDK22 APIs for Valhalla#18327
keithc-ca merged 1 commit into
eclipse-openj9:masterfrom
JasonFengJ9:fixvalhalla

Conversation

@JasonFengJ9

@JasonFengJ9 JasonFengJ9 commented Oct 23, 2023

Copy link
Copy Markdown
Member

Disable latest JDK22 APIs for Valhalla

Valhalla doesn't support the latest JDK22 APIs:
JavaLangAccess.ensureNativeAccess(Module mod, Class<?> clsOwner, String methodName, Class<?> clsCaller);
JavaLangAccess.bytesCompatible(String string, Charset charset);
JavaLangAccess.copyToSegmentRaw(String string, MemorySegment segment, long offset);
LinkerOptions.isCritical().

FYI @hangshao0

Signed-off-by: Jason Feng fengj@ca.ibm.com

@JasonFengJ9 JasonFengJ9 added comp:vm project:valhalla Used to track Project Valhalla related work labels Oct 23, 2023
@JasonFengJ9
JasonFengJ9 requested a review from keithc-ca October 23, 2023 11:57
Comment thread jcl/src/java.base/share/classes/java/lang/Access.java Outdated
Valhalla doesn't support latest JDK22 APIs:
JavaLangAccess.ensureNativeAccess(Module mod, Class<?> clsOwner, String
methodName, Class<?> clsCaller);
JavaLangAccess.bytesCompatible(String string, Charset charset);
JavaLangAccess.copyToSegmentRaw(String string, MemorySegment segment,
long offset);
LinkerOptions.isCritical().

Signed-off-by: Jason Feng <fengj@ca.ibm.com>

@hangshao0 hangshao0 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM.

@hangshao0

Copy link
Copy Markdown
Contributor

jenkins compile winval jdknext

@keithc-ca
keithc-ca merged commit 2eb0050 into eclipse-openj9:master Oct 23, 2023
@JasonFengJ9
JasonFengJ9 deleted the fixvalhalla branch October 23, 2023 20:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp:vm project:valhalla Used to track Project Valhalla related work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants