-
Notifications
You must be signed in to change notification settings - Fork 396
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
Expose alloc cache size API #7263
Conversation
@dmitripivkine please, review |
427d904
to
1a93a9d
Compare
@babsingh Would you please help to merge this? |
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.
Thanks for the reminder. Minor formatting nit: @return
description should be on a new line. Otherwise, LGTM.
Expose remaining and refresh size APIs for ObjectAllocationInterface, and implement it for TLHAllocationInterface. While this values are functionally needed only internally within the class, they will be used for stats reporting purposes outisde the class, hence need to be public. Signed-off-by: Aleksandar Micic <Aleksandar_Micic@ca.ibm.com>
1a93a9d
to
55404bf
Compare
@babsingh have a look again |
jenkins build all |
jenkins build win |
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.
OSX PR build failed due to a known and unrelated issue: #7181.
Expose remaining and refresh size APIs for ObjectAllocationInterface,
and implement it for TLHAllocationInterface.
While this values are functionally needed only internally within the
class, they will be used for stats reporting purposes outisde the class,
hence need to be public.