Skip to content

Feature/optimizations - #5

Open
marc2911 wants to merge 2 commits into
ChristianSauer:masterfrom
marc2911:feature/optimizations
Open

Feature/optimizations#5
marc2911 wants to merge 2 commits into
ChristianSauer:masterfrom
marc2911:feature/optimizations

Conversation

@marc2911

Copy link
Copy Markdown

Optimizations: Avoid redundant extra work

1. optimized src gen pipeline to not recompile all files whenever a single one changes

found out via Debug.Log() that we actually regenerate all files whenever only a single one changes

  • introduced some container model that can compare the syntax to tell Roselyn whether that file actually changed
  • comparing the whole syntax string is potentially also expensive but probably less so compared to the I/O overhead of generating files
  • but at least we dont miss any changes that way! future work can make this more fine-grained to check only the declarations we care about.. i couldnt make that reliably work

2. removed redundant enumerations

fix double enumeration issue when calling

enumerable.ToList().ForEach()

to

foreach(var x in enumerable) {}

@marc2911
marc2911 marked this pull request as ready for review August 23, 2026 11:51
@marc2911
marc2911 force-pushed the feature/optimizations branch from 567ae93 to 38e933e Compare August 23, 2026 12:07
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