-
Notifications
You must be signed in to change notification settings - Fork 729
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
Ensure inlined method classes are dependencies #20730
Conversation
Attn @mpirvu. This does not seem to affect the failure rate much, if at all, because these classes will generally be mentioned in other relo records and so be captured as dependencies there. It's still good to make sure they're counted, I think. |
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 |
AIX compilation error:
|
I'll fix that. I also see that I missed adding a dependency in I tested #20770 with this change added to make sure the failures weren't a result of the lack of dependencies. I'll double-check that PR with this last dependency added. |
Signed-off-by: Christian Despres <[email protected]>
This should be fixed. Adding that last dependency did not seem to affect anything. |
jenkins test sanity all jdk21 |
No description provided.