rust-lang/rust-analyzer

Expression lowering uses unreasonable amount of stack space

Offen

#5.317 geöffnet am 11.07.2020

Auf GitHub ansehenContributor Guide
 (4 Kommentare) (0 Reaktionen) (1 zugewiesene Person)Rust (1.726 Forks)batch import
A-perfC-enhancementE-hardS-actionablefungood first issue

Repository-Metriken

Stars
 (14.924 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 11T 5h) (96 gemergte PRs in 30 T)

Beschreibung

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!

Contributor Guide

Research-Richtung
Profilieren Sie die Expression Lowering Funktion, um die Stack Nutzung zu messen, und refaktorisieren Sie dann die große Match Anweisung in separate Funktionen oder verwenden Sie Box, um die Stack Größe zu reduzieren.
Tech Stack
rust
Domain
compilersbackenddeveloper experience
Issue Type
Performance
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
RustGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60