facebookarchive/prepack

Use a much smaller upper limit for the size of the values domain of a binary operation

Open

#1000 aperta il 26 set 2017

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)JavaScript (520 fork)batch import
abstractenhancementhelp wanted

Metriche repository

Star
 (14.268 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

The maximum number of values that will be tracked in the ValuesDomain as the result of a binary operation or logical operation is currently 10 000. This is unlikely to be helpful. Add some instrumentation to get hold of the actual distribution of the sizes of value domains, given some relevant benchmarks, and then limit the size of the domain to the 99 percentile.

Guida contributor

Direzione di ricerca
Analizza il codice che calcola il dominio dei valori per le operazioni binarie, aggiungi strumentazione per registrare la distribuzione delle dimensioni, esegui benchmark, calcola il 99° percentile e regola il limite di conseguenza.
Tech stack
javascript
Dominio
compilers
Tipo issue
Performance
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScriptGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60