facebookarchive/prepack

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

Open

#1 000 ouverte le 26 sept. 2017

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)JavaScript (520 forks)batch import
abstractenhancementhelp wanted

Métriques du dépôt

Stars
 (14 268 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Analysez le code qui calcule le domaine de valeurs pour les opérations binaires, ajoutez une instrumentation pour enregistrer la distribution des tailles, exécutez des benchmarks, calculez le 99e percentile et ajustez la limite en conséquence.
Stack technique
javascript
Domaine
compilers
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.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
JavaScriptGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60