rust-lang/rust-analyzer

Expression lowering uses unreasonable amount of stack space

Open

#5 317 ouverte le 11 juil. 2020

Voir sur GitHub
 (3 commentaires) (0 réactions) (1 assigné)Rust (1 726 forks)batch import
A-perfC-enhancementE-hardS-actionablefungood first issue

Métriques du dépôt

Stars
 (14 924 stars)
Métriques de merge PR
 (Merge moyen 3j 11h) (113 PRs mergées en 30 j)

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!

Guide contributeur

Direction de recherche
Étudiez le code de réduction d'expressions dans le dépôt rust analyzer, en particulier l'énorme instruction match qui consomme un espace de pile excessif. Examinez le commit référencé (12d52a7) dans la PR #5316 pour les méthodes d'observation de l'utilisation de la pile. Envisagez de diviser le match en fonctions distinctes ou d'utiliser des fermetures boxées pour réduire la profondeur de la pile. Recherchez tout commentaire ou discussion existant sur le problème pour un contexte supplémentaire.
Stack technique
rust
Domaine
performancetooling
Type d'issue
Performance
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
5
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
3-5 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Bloquée
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
Rust programmingcompiler internalsexpression lowering
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
10