Skip to content

GC finalization cleanup - #91129

Merged
markples merged 7 commits into
dotnet:mainfrom
markples:finalize-cleanup
Sep 8, 2023
Merged

GC finalization cleanup#91129
markples merged 7 commits into
dotnet:mainfrom
markples:finalize-cleanup

Conversation

@markples

@markples markples commented Aug 25, 2023

Copy link
Copy Markdown
Contributor

Pulled from #91029

mangod9 and others added 3 commits August 14, 2023 15:46
* adding clrgc variant with regions enabled

* only build regions for 64 bit platforms

* rename to clrgcexp for experimental
@ghost ghost added the area-GC-coreclr label Aug 25, 2023
@ghost ghost assigned markples Aug 25, 2023
@ghost

ghost commented Aug 25, 2023

Copy link
Copy Markdown

Tagging subscribers to this area: @dotnet/gc
See info in area-owners.md if you want to be subscribed.

Issue Details

null

Author: markples
Assignees: -
Labels:

area-GC-coreclr

Milestone: -

@markples
markples marked this pull request as ready for review August 25, 2023 23:37
@markples

markples commented Aug 25, 2023

Copy link
Copy Markdown
Contributor Author

I'm still learning how to test one of the DATAS codepaths and diag codepaths, but otherwise I ran short stretches of reliability framework on wks and svr and CI is clean. I also verified that this doesn't improve the finalization benchmark performance in any way, which would have been quite surprising.

@markples

markples commented Sep 7, 2023

Copy link
Copy Markdown
Contributor Author

/azp run runtime

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@markples

markples commented Sep 8, 2023

Copy link
Copy Markdown
Contributor Author

Failures are networking and a timeout, and the set is different from a previous run.

@markples
markples merged commit 9ae1a7c into dotnet:main Sep 8, 2023
@ghost ghost locked as resolved and limited conversation to collaborators Oct 8, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants