-
Notifications
You must be signed in to change notification settings - Fork 733
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
Allowing JFR to get J9Class of hidden and anonymous classes #20979
base: master
Are you sure you want to change the base?
Conversation
@tajila can you please review? |
runtime/vm/KeyHashTable.c
Outdated
@@ -333,7 +342,7 @@ hashClassTableNew(J9JavaVM *javaVM, U_32 initialSize) | |||
} | |||
|
|||
J9Class * | |||
hashClassTableAt(J9ClassLoader *classLoader, U_8 *className, UDATA classNameLength) | |||
hashClassTableAt(J9ClassLoader *classLoader, U_8 *className, UDATA classNameLength, BOOLEAN ignoreHiddenClass) |
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.
I should have mentioned this earlier, but you can make:
hashClassTableAtImpl(J9ClassLoader *classLoader, U_8 *className, UDATA classNameLength, BOOLEAN ignoreHiddenClass) { }
and then
hashClassTableAt(J9ClassLoader *classLoader, U_8 *className, UDATA classNameLength)
{
return hashClassTableAtImpl(classLoader, className, classNameLength, TRUE);
}
So you dont need to update all the places its used.
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.
Updated my PR. I added hashClassTableAtImpl
to vm_api.h
, is that what I'm supposed to do?
Signed-off-by: Gengchen Tuo <[email protected]>
jenkins test sanity,extended.functional,extended.opendk,sanity.special alinux64 jdk17 |
@thallium There are crashes in the tests:
|
No description provided.