-
Notifications
You must be signed in to change notification settings - Fork 706
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
Update GC DDR to Remove Unused GC MemoryPoolHybrid #17106
Conversation
- Delete MemoryPoolHybrid.hpp/cpp files - Remove Hybrid Pool specific _heapFreeListCountExtended field from base class MM_MemoryPoolSplitAddressOrderedListBase Depends on: eclipse-openj9/openj9#17106 Signed-off-by: Salman Rana <salman.rana@ibm.com>
@dmitripivkine @amicic please have a look |
MemoryPoolHybrid is being removed in OMR, all references to it must also be removed in OpenJ9. Signed-off-by: Salman Rana <salman.rana@ibm.com>
- Delete MemoryPoolHybrid.hpp/cpp files - Remove Hybrid Pool specific _heapFreeListCountExtended field from base class MM_MemoryPoolSplitAddressOrderedListBase Depends on: eclipse-openj9/openj9#17106 Signed-off-by: Salman Rana <salman.rana@ibm.com>
cc: @keithc-ca |
Running some personal builds to verify this won't break anything, will update the PR once everything looks good. |
class DDR_MM_MemoryPoolHybrid : public MM_MemoryPoolHybrid | ||
{ | ||
public: | ||
const char * ddrHelper(); | ||
}; | ||
|
||
const char * | ||
DDR_MM_MemoryPoolHybrid::ddrHelper() | ||
{ | ||
return this->_typeId; | ||
} |
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.
My only potential concern is if this was required to force the compiler to properly describe some ancestor class of MM_MemoryPoolHybrid
. We'll need to pay close attention to any effects this has on AIX builds.
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.
All builds pass, no issues (Axxon: /build_info.php?build_id=48521)
Jenkins compile aix jdk8,jdk17 |
GC MemoryPoolHybrid is being removed in OMR, all references to it must also be removed in OpenJ9.
Signed-off-by: Salman Rana salman.rana@ibm.com