-
Notifications
You must be signed in to change notification settings - Fork 710
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
Disable VarHandleMethodType lookup table code for OJDK MH #13858
Conversation
c709535
to
d9c5fae
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm. commits need to be squashed. @nbhuiyan can you review the JIT changes?
Ran the openjdk jck system sanity tests for jdk11 and 17 and all passed. |
J9RomClass.varHandleMethodTypeLookupTable is not used in the OJDK method handle implementation as varhandles are treated as method handles and use the invoke cache instead. Closes: eclipse-openj9#11580 Signed-off-by: Eric Yang <eric.yang@ibm.com>
1530c64
to
f75956b
Compare
Jenkins test sanity,extended win jdk8,jdk17 |
@jdmpapin Please review the JIT changes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
JIT changes LGTM
J9RomClass.varHandleMethodTypeLookupTable is not used in the OJDK method
handle implementation as varhandles are treated as method handles and
use the invoke cache instead.
Closes: #11580
Signed-off-by: Eric Yang eric.yang@ibm.com