Skip to content

Upgrade to .NET 8#1077

Closed
ScrubN wants to merge 10 commits intolay295:masterfrom
ScrubN:net-8
Closed

Upgrade to .NET 8#1077
ScrubN wants to merge 10 commits intolay295:masterfrom
ScrubN:net-8

Conversation

@ScrubN
Copy link
Collaborator

@ScrubN ScrubN commented May 27, 2024

Pros:

  • More language features
  • Native folder browser dialog
  • Native Unix permissions handling (Mono.Posix can be removed)
  • Source generated regex = noticeably faster startup times

Cons:

  • For some stupid reason parts of the standard library that are used heavily by chat renders are significantly slower, leading to 30-100% slower render times.

@ScrubN
Copy link
Collaborator Author

ScrubN commented May 27, 2024

I'm going to continue to investigate the root of the slowdowns. I don't know if I'll find anything, but hopefully I can.

@ScrubN
Copy link
Collaborator Author

ScrubN commented Jun 4, 2024

In a barebones project, it seems that writing the bytes to stdin is not where the regression is.

@ScrubN ScrubN marked this pull request as draft June 8, 2024 02:14
@ScrubN ScrubN force-pushed the net-8 branch 2 times, most recently from e01ad57 to 9d6bdc5 Compare June 8, 2024 22:09
@ScrubN
Copy link
Collaborator Author

ScrubN commented Jan 22, 2025

The slowdown is caused by the new GC. Forcing clrgc.dll with environment variables solves the performance regressions.

@ScrubN
Copy link
Collaborator Author

ScrubN commented Feb 3, 2026

Superseded by #1557

@ScrubN ScrubN closed this Feb 3, 2026
@ScrubN ScrubN deleted the net-8 branch February 4, 2026 21:37
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.

1 participant