-
Notifications
You must be signed in to change notification settings - Fork 722
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
Add support for class lookup by name in CP for JITServer/AOT #20686
Conversation
Attn @mpirvu. |
175983c
to
cbc15f7
Compare
cbc15f7
to
427a956
Compare
The getClassFromSignature() query using a J9ConstantPool is now supported in all frontends. Signed-off-by: Christian Despres <[email protected]>
Signed-off-by: Christian Despres <[email protected]>
427a956
to
f8a959b
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
jenkins test sanity all jdk21 |
Test failure on s390x was in
I think it's #17975. |
The The
There have been some "Agent communication error" failures in this test recently (#19962), but I'm not sure if they're related. There is a |
I started a 10x grinder for the AIX failure here: https://openj9-jenkins.osuosl.org/job/Grinder/4006/ |
Three failures in the AIX grinder were
matching #19962, and the other seven were successful. |
I had another 10 iteration grinder with the nightly build: https://openj9-jenkins.osuosl.org/job/Grinder/4007/ |
Classes can now be looked up by signature in a constant pool when compiling AOT or remote methods. An SVM assertion failure relating to shadow symbols has also been fixed using this new support.
Fixes: #20669