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

ValueTypes: add new java.lang.Class API methods JIT implementation #13695

Open
tajila opened this issue Oct 13, 2021 · 4 comments · May be fixed by #20330
Open

ValueTypes: add new java.lang.Class API methods JIT implementation #13695

tajila opened this issue Oct 13, 2021 · 4 comments · May be fixed by #20330
Labels
comp:jit project:valhalla Used to track Project Valhalla related work

Comments

@tajila
Copy link
Contributor

tajila commented Oct 13, 2021

    public Class<?> asPrimaryType();
    public Class<?> asValueType();
    public boolean isPrimaryType();
    public boolean isValueType(); 
    public boolean isPrimitiveClass()

See #13615

@tajila tajila added comp:jit project:valhalla Used to track Project Valhalla related work labels Oct 13, 2021
@hangshao0
Copy link
Contributor

A bit update on the APIs: #13615 (comment)

@theresa-m
Copy link
Contributor

These methods have mostly been removed (see #19736). The remaining are:

  • Class.isValue
  • Class.asNullRestrictedType

@hangshao0
Copy link
Contributor

I don't see Class.asNullRestrictedType here:

https://github.com/ibmruntimes/openj9-openjdk-jdk.valuetypes/blob/openj9/src/java.base/share/classes/java/lang/Class.java

Where did you see that ? If it is gone, this can be closed.

@theresa-m
Copy link
Contributor

You are right. I'll remove it from OpenJ9's Class.java and close this then.

@theresa-m theresa-m linked a pull request Oct 10, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comp:jit project:valhalla Used to track Project Valhalla related work
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants