Uh oh!
There was an error while loading. Please reload this page.
Remove obsolete ILLink dependency analyzer project and references - #125242
Merged
Conversation
Co-authored-by: sbomer <787361+sbomer@users.noreply.github.com>
CopilotAI
changed the title
[WIP] Remove the unused ILLink analyzer and referencesRemove obsolete ILLink dependency analyzer project and referencesMar 5, 2026
sbomer
approved these changes
Mar 6, 2026
Contributor
There was a problem hiding this comment.
Pull request overview
Removes the obsolete ILLink dependency analyzer tool from the repository and cleans up remaining build/solution/docs wiring so the ILLink toolset reflects only actively maintained components.
Changes:
- Removed the analyzer project from the
tools.illinkbuild subset. - Removed the analyzer project from ILLink solution (
.slnx) files. - Deleted the analyzer project sources and removed the corresponding documentation section/link.
Reviewed changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| eng/Subsets.props | Drops src/tools/illink/src/analyzer/analyzer.csproj from the tools.illink subset build graph. |
| src/tools/illink/illink.slnx | Removes the analyzer project entry from the ILLink solution definition. |
| src/tools/illink/trimming.slnx | Removes the analyzer project entry from the trimming solution definition. |
| src/tools/illink/README.md | Removes the “Dependencies Analyzer” documentation section/link. |
| src/tools/illink/src/analyzer/analyzer.csproj | Deleted obsolete analyzer project file. |
| src/tools/illink/src/analyzer/README.md | Deleted analyzer-specific README. |
| src/tools/illink/src/analyzer/Main.cs | Deleted analyzer CLI entrypoint. |
| src/tools/illink/src/analyzer/ConsoleDependencyGraph.cs | Deleted analyzer console output implementation. |
| src/tools/illink/src/analyzer/LinkerAnalyzerCore/DependencyGraph.cs | Deleted analyzer core dependency graph implementation. |
| src/tools/illink/src/analyzer/LinkerAnalyzerCore/SpaceAnalyzer.cs | Deleted analyzer space analysis implementation. |
jtschuster
approved these changes
Mar 6, 2026
sbomer
enabled auto-merge (squash)
March 6, 2026 17:16
sbomer
commented
Mar 6, 2026
Member
/ba-g "timeout/dead-letter" |
agocke
reviewed
Mar 6, 2026
| <ProjectToBuild Include="$(ToolsProjectRoot)illink\src\analyzer\analyzer.csproj" Category="tools" /> | ||
| <ProjectToBuild Include="$(ToolsProjectRoot)illink\src\ILLink.RoslynAnalyzer\ILLink.RoslynAnalyzer.csproj" Category="tools" /> | ||
| <ProjectToBuild Include="$(ToolsProjectRoot)illink\src\linker\ref\Mono.Linker.csproj" Category="tools" /> | ||
| <ProjectToBuild Include="$(ToolsProjectRoot)illink\src\tlens\tlens.csproj" Category="tools" /> |
Member
There was a problem hiding this comment.
What's this? I don't think I've ever seen tlens before
Member
There was a problem hiding this comment.
I've never used it myself, seems like another one we could delete.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This change removes the unused ILLink dependency analyzer under
src/tools/illink/src/analyzerand eliminates all repository wiring that still referenced it. The ILLink toolset surface now reflects only actively maintained components.Build graph cleanup
eng/Subsets.props(tools.illinksubset no longer buildssrc/analyzer/analyzer.csproj).Solution cleanup
src/analyzer/analyzer.csprojfrom:src/tools/illink/illink.slnxsrc/tools/illink/trimming.slnxSource removal
src/tools/illink/src/analyzer/(project file, core implementation, and analyzer-specific README).Docs alignment
src/tools/illink/README.mdto remove the “Dependencies Analyzer” section/link to the removed component.💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.