facebookarchive/prepack

Collect statistics that help understand Prepack effects

Open

#934 geöffnet am 26. Aug. 2017

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (520 Forks)batch import
enhancementhelp wanted

Repository-Metriken

Stars
 (14.268 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

Prepack runs the global code, and we could collect some runtime information to help assess what Prepack does:

  • How many objects get created when running the interpreter? (And how many survive and get serialized?)
  • How many object properties get updated? (And how many are serialized?)
  • How many times does each type of AST node get evaluated? (Pretty much all of those computations, except for some literals, object allocations, property updates, will get eliminated by Prepack.)

When forking and merging control-flow, compute the maximum value for each explored case.

Contributor Guide

Research-Richtung
Die Implementierung erfordert die Änderung des Interpreters (wahrscheinlich in src/interpreter.js), um die Objekterstellung und Eigenschaftsaktualisierungen zu instrumentieren. Die Serialisierungslogik in src/serializer.js muss verfolgen, welche Objekte serialisiert werden. Die AST Auswertung (src/eval.js) sollte jeden ausgewerteten Knotentyp zählen. Die Kontrollflusszusammenführung (src/flow.js) muss die Maximalwerte über alle Zweige berechnen. Bestehende PRs oder Kommentare bieten keine Anleitung.
Tech Stack
javascript
Domain
toolingperformance
Issue Type
Funktion
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.
1-2 Tage
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
JavaScriptNode.js
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
40