dotnet/roslyn

Remove extraneous branch for empty strings in interpolated string handler CFG

Aberta

#57.611 aberto em 5 de nov. de 2021

Ver no GitHubGuia do colaborador
 (0 comentário) (0 reação) (1 responsável)C# (4.257 forks)batch import
Area-CompilersBugFeature - IOperationFeature - Interpolated String Improvementshelp wanted

Métricas do repositório

Stars
 (20.414 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

When an interpolated string handler is created for an empty string, and the constructor of the handler uses an out bool success parameter, we generate an unnecessary branch in the CFG for the code, demonstrated in the InterpolatedStringHandlerConversionFlow_16 test. We could take the approach Aleksey detailed in his review to remove that branch, but we will have to wire a CreateDiscardSymbol method through the semantic model to be able to create an IDiscardOperation for the parameter value. This is presently undone.

Guia do colaborador

Direção de pesquisa
Estude o teste InterpolatedStringHandlerConversionFlow 16 para entender a geração atual do CFG. Implemente o CreateDiscardSymbol no modelo semântico e use o para evitar o branch desnecessário, seguindo a sugestão da revisão.
Pilha de tecnologia
csharp
Domain
None
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-2 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C#Roslyn compiler internals
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
40