Improve caching in XmlToDescriptionGenerator #1010
Conversation
| ImmutableArray<string> TypeParameters, ImmutableArray<ParameterInfo> Parameters, | ||
| string? MethodDescription, string? ReturnDescription, ImmutableArray<DiagnosticInfo> Diagnostics) |
There was a problem hiding this comment.
Immutable array is likely to hurt equality here. I personally always reuse EquatableArray from @Sergio0694 for this. https://github.com/Sergio0694/ComputeSharp/blob/main/src/ComputeSharp.SourceGeneration/Helpers/EquatableArray%7BT%7D.cs
| private readonly record struct DiagnosticInfo(string Id, Location? Location, params object?[] MessageArgs); | ||
|
|
||
| /// <summary>Holds extracted XML documentation for a method.</summary> | ||
| private sealed record XmlDocumentation(string MethodDescription, string Returns, Dictionary<string, string> Parameters); |
There was a problem hiding this comment.
This Dictionary should still be a problem, isn't it?
| private readonly record struct MethodToGenerate(MethodDeclarationSyntax MethodDeclaration, IMethodSymbol MethodSymbol); | ||
| private readonly record struct TypeDeclarationInfo(string Name, string TypeKeyword); | ||
|
|
||
| private readonly record struct DiagnosticInfo(string Id, Location? Location, params object?[] MessageArgs); |
There was a problem hiding this comment.
This array here is likely to be a problem as well?
| // Combine with compilation to get well-known type symbols. | ||
| var compilationAndMethods = context.CompilationProvider.Combine(allMethods); | ||
| // Report diagnostics. | ||
| context.RegisterSourceOutput(allMethods, static (spc, methods) => |
There was a problem hiding this comment.
This separation of diagnostic reporting into a separate callback is an interesting approach. I think it might provide a viable workaround for dotnet/runtime#92509. Curious if you've applied this approach elsewhere before, or if there are any drawbacks to it.
Closes #1003