Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Linker dependency framework #101277

Merged
merged 42 commits into from
May 6, 2024
Merged
Changes from 1 commit
Commits
Show all changes
42 commits
Select commit Hold shift + click to select a range
afe8990
MarkStep runs inside dependency framework
vitek-karas Sep 13, 2023
4a92aaf
Merge branch 'main' of https://github.com/dotnet/runtime into LinkerD…
jtschuster Apr 19, 2024
76fe923
Put MarkType into DependencyFramework
jtschuster Apr 19, 2024
b3ccd66
Leave MarkType in MarkStep, use captured MarkStep to call it
jtschuster Apr 19, 2024
eb5e333
Format
jtschuster Apr 19, 2024
e1270bb
Use DF for TypeDefinition and MethodDefinition
jtschuster Apr 19, 2024
2d12ade
Rename MarkTypeImpl to ProcessType and clean up code
jtschuster Apr 20, 2024
1dc80db
Clean up code a bit more
jtschuster Apr 20, 2024
991f7a0
Use ProjectReference for DAF, and undo method moves
jtschuster Apr 22, 2024
069f47b
Merge branch 'main' of https://github.com/dotnet/runtime into LinkerD…
jtschuster Apr 22, 2024
c1670eb
Don't have additional virtualmethods processing
jtschuster Apr 22, 2024
8382e68
Merge branch 'main' of https://github.com/dotnet/runtime into LinkerD…
jtschuster Apr 22, 2024
5bf5ece
Remove old header comment copied from MarkStep
jtschuster Apr 22, 2024
e6edbce
Remove nullable directives from ilc files
jtschuster Apr 22, 2024
10a5c57
Sign DAF instead of nowarn on trimmer
jtschuster Apr 22, 2024
166d2d5
Re-add doc comment; Make types internal
jtschuster Apr 22, 2024
01f3a79
Undo Queue changes, fix visibility issues
jtschuster Apr 23, 2024
098f7bc
Rename TypeDefinitionDependencyNode file to match class
jtschuster Apr 23, 2024
25ec8ce
PR feedback:
jtschuster Apr 23, 2024
9ef1343
PR Feedback:
jtschuster Apr 25, 2024
d50d70c
Use static lambda for TypeDefinitionNode creation
jtschuster Apr 25, 2024
dfe51f2
Only warn on duplicate members within a single descriptors file
jtschuster Apr 25, 2024
afedfb7
Only warn on duplicate members within a single descriptors file
jtschuster Apr 25, 2024
a600348
Make duplicate preserve an info message
jtschuster Apr 26, 2024
8d642a5
Merge branch 'main' of https://github.com/dotnet/runtime into LinkerD…
jtschuster Apr 26, 2024
a0ea640
Merge branch 'main' of https://github.com/dotnet/runtime into ILLinkD…
jtschuster Apr 26, 2024
2bb99d7
Merge branch 'ILLinkDescriptionDuplicatePreserve' into LinkerDependen…
jtschuster Apr 26, 2024
b27bc04
Update expectations in xml warnings tests
jtschuster Apr 26, 2024
3394758
Merge branch 'ILLinkDescriptionDuplicatePreserve' into LinkerDependen…
jtschuster Apr 26, 2024
1abe4b3
Make method internal for API compat
jtschuster Apr 27, 2024
8cbedd1
Merge branch 'ILLinkDescriptionDuplicatePreserve' into LinkerDependen…
jtschuster Apr 27, 2024
df7a522
Merge branch 'main' into LinkerDependencyFramework
jtschuster Apr 30, 2024
28bf0be
Merge branch 'main' into LinkerDependencyFramework
jtschuster May 1, 2024
3ca9f9a
PR Feedback:
jtschuster May 2, 2024
830f23e
Merge branch 'LinkerDependencyFramework' of https://github.com/jtschu…
jtschuster May 2, 2024
3001124
Add space before method call parentheses
jtschuster May 2, 2024
f003f74
Don't pass TargetOS and TargetArch to ILLink.Tasks when using live il…
jtschuster May 3, 2024
a0f6861
Merge branch 'main' into LinkerDependencyFramework
jtschuster May 3, 2024
d358b42
Update comment in liveILLink.targets
jtschuster May 3, 2024
0fab4e3
Merge branch 'LinkerDependencyFramework' of https://github.com/jtschu…
jtschuster May 3, 2024
5e124b5
Don't append platform to output dirs
jtschuster May 3, 2024
855b109
Merge branch 'main' into LinkerDependencyFramework
jtschuster May 6, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Add space before method call parentheses
  • Loading branch information
jtschuster committed May 2, 2024
commit 30011248d67b14a8ef7cd5dbdd8e1e9c868aaf94
4 changes: 2 additions & 2 deletions src/tools/illink/src/linker/Linker.Steps/MarkStep.cs
Original file line number Diff line number Diff line change
@@ -2016,7 +2016,7 @@ internal void MarkStaticConstructorVisibleToReflection (TypeDefinition type, in
if (type.Scope is ModuleDefinition module)
MarkModule (module, new DependencyInfo (DependencyKind.ScopeOfType, type));

_dependencyGraph.AddRoot (_nodeFactory.GetTypeNode (type), Enum.GetName(reason.Kind));
_dependencyGraph.AddRoot (_nodeFactory.GetTypeNode (type), Enum.GetName (reason.Kind));
return type;
}

@@ -3035,7 +3035,7 @@ void MarkMethodCollection (IList<MethodDefinition> methods, in DependencyInfo re
// We will only enqueue a method to be processed if it hasn't been processed yet.
if (!CheckProcessed (method))
_completed = false;
_dependencyGraph.AddRoot (_nodeFactory.GetMethodDefinitionNode (method, reason), Enum.GetName(reason.Kind));
_dependencyGraph.AddRoot (_nodeFactory.GetMethodDefinitionNode (method, reason), Enum.GetName (reason.Kind));

return method;
}
Loading