-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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 clrgcexp to NetCoreApp #97401
Add clrgcexp to NetCoreApp #97401
Conversation
clrgcexp is standalone GC with Regions enabled and will also be used for experimental features.
Tagging subscribers to this area: @dotnet/gc Issue Detailsclrgcexp is standalone GC with Regions enabled and will also be used for experimental features.
|
src/coreclr/gc/CMakeLists.txt
Outdated
@@ -135,3 +135,5 @@ include_directories(${CMAKE_CURRENT_SOURCE_DIR}) | |||
include_directories(${CMAKE_CURRENT_SOURCE_DIR}/env) | |||
|
|||
install_clr(TARGETS clrgc DESTINATIONS . sharedFramework COMPONENT runtime) | |||
install_clr(TARGETS clrgcexp DESTINATIONS . sharedFramework COMPONENT runtime) |
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.
Are the two install_clr
lines above (113 and 120) subsumed by these two? Regardless, should this new line be 64-bit-only?
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.
yeah needs to be 64-bit only. Have updated it now.
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!
clrgcexp is standalone GC with Regions enabled and will also be used for experimental features.