Skip to content

update --times logic #13751

Merged
vzarytovskii merged 6 commits into
dotnet:mainfrom
dsyme:gc4
Aug 24, 2022
Merged

update --times logic #13751
vzarytovskii merged 6 commits into
dotnet:mainfrom
dsyme:gc4

Conversation

@dsyme

@dsyme dsyme commented Aug 22, 2022

Copy link
Copy Markdown
Contributor

Based on recent testing

  • Update --times option to report wall-clock time, not total user CPU time , since the latter isn't really helpful in multiproc situations, and almost misled us into thinking GC Server was worse, when it actually gives better wallclock performance
  • Remove tests for this internal option

@vzarytovskii @KevinRansom Personally I'm happy with just removing the tests for this internal options, I can re-instate them if you like

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants