rust-lang/rust-analyzer

Expression lowering uses unreasonable amount of stack space

Open

#5.317 geöffnet am 11. Juli 2020

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

Repository-Metriken

Stars
 (14.924 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 3T 11h) (113 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
Untersuche den Ausdrucks Lowering Code im rust analyzer Repository, insbesondere die riesige match Anweisung, die übermäßig viel Stack Speicher verbraucht. Überprüfe den referenzierten Commit (12d52a7) in PR #5316 auf Methoden zur Beobachtung der Stack Nutzung. Erwäge, den match in separate Funktionen aufzuteilen oder boxed Closures zu verwenden, um die Stack Tiefe zu reduzieren. Suche nach vorhandenen Kommentaren oder Diskussionen zu diesem Problem für weiteren Kontext.
Tech Stack
rust
Domain
performancetooling
Issue Type
Performance
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
5
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
3-5 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Blockiert
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
Rust programmingcompiler internalsexpression lowering
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
10