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.

Sign up for GitHub

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

Jump to bottom

ValueTypes: checkcast behaviour #12130

Closed
tajila opened this issue Mar 4, 2021 · 3 comments
Closed

ValueTypes: checkcast behaviour #12130

tajila opened this issue Mar 4, 2021 · 3 comments
Labels
comp:vm project:valhalla Used to track Project Valhalla related work

Comments

@tajila
Copy link
Contributor

tajila commented Mar 4, 2021

The following bytecodes trigger NoClassDef exception on J9. However on the RI it throws an NPE.

Code:
       0: aconst_null
       1: checkcast     #54                 // class "QClassDoesNotExist;"
       4: areturn

We need to change the behaviour so it matches RI, we also need to make sure we do the right thing when classref is "QClassDoesNotExist;" and reciever is not NULL
@tajila tajila added comp:vm project:valhalla Used to track Project Valhalla related work labels Mar 4, 2021
@gacholio
Copy link
Contributor

gacholio commented Mar 5, 2021

Are we sure the RI behaviour is correct?

@theresa-m
Copy link
Contributor

This can be closed since Q types are gone.

@github-actions GitHub Actions
Copy link

Issue Number: 12130
Status: Closed
Actual Components: comp:vm, project:valhalla
Actual Assignees: No one :(
PR Assignees: No one :(

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
comp:vm project:valhalla Used to track Project Valhalla related work
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@tajila @theresa-m @gacholio