rust-lang/rust-analyzer

Expression lowering uses unreasonable amount of stack space

Open

#5.317 aberto em 11 de jul. de 2020

Ver no GitHub
 (3 comments) (0 reactions) (1 assignee)Rust (1.726 forks)batch import
A-perfC-enhancementE-hardS-actionablefungood first issue

Métricas do repositório

Stars
 (14.924 stars)
Métricas de merge de PR
 (Mesclagem média 3d 11h) (113 fundiu PRs em 30d)

Description

Our lowering code for expressions is a single giant match, and looks like it consumes a lot of stack space, even in release mode.

See https://github.com/rust-analyzer/rust-analyzer/pull/5316/commits/12d52a726a1e6ca0f6b08343d13d0f6b50d94d8c for some ways to observe this.

It would be nice to optimize this!

Guia do colaborador

Direção de pesquisa
Investigue o código de rebaixamento de expressões no repositório rust analyzer, especificamente a enorme instrução match que consome espaço excessivo na pilha. Revise o commit referenciado (12d52a7) no PR #5316 para métodos de observar o uso da pilha. Considere dividir o match em funções separadas ou usar closures em caixa para reduzir a profundidade da pilha. Procure por quaisquer comentários ou discussões existentes sobre o problema para contexto adicional.
Pilha de tecnologia
rust
Domain
performancetooling
Tipo Issue
Performance
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
5
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
3-5 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Bloqueado
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Principalmente claro
Prerequisites
Rust programmingcompiler internalsexpression lowering
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
10