I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

Thanks.

Optimize DateTimeOffset

Can you share performance numbers? What are the wins and where are they coming from?

Cleanup

Can you remove the style-only changes? It's not clear to me what's here for optimization reasons and what's just style.

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

Can you share performance numbers? What are the wins and where are they coming from?

@pentp For the performance numbers, we'll want to see output from BenchmarkDotNet for the affected microbenchmarks, comparing the results with and without these changes. If needed, you can refer to dotnet/performance for more info.

Thanks!

@pentp are you still working on this?

No particular rush, just wanted to ensure that was the case since many people took a break over the holidays. There are still several pending comments above that would need to be resolved.

This pull request has been automatically marked no recent activity because it has not had any activity for 14 days. It will be closed if no further activity occurs within 14 more days. Any new comment (by anyone, not necessarily the author) will remove no recent activity.

@tannergooding I took a longer break from non-work coding. Will start getting back on track with my open PRs this weekend probably.

This pull request has been automatically marked no-recent-activity because it has not had any activity for 14 days. It will be closed if no further activity occurs within 14 more days. Any new comment (by anyone, not necessarily the author) will remove no-recent-activity.

This pull request will now be closed since it had been marked no-recent-activity but received no further activity in the past 14 days. It is still possible to reopen or comment on the pull request, but please note that it will be locked if it remains inactive for another 30 days.