facebookarchive/prepack

Prepack emits if-statements with no consequent or alternate code

Open

#2 220 ouverte le 6 juil. 2018

Voir sur GitHub
 (3 commentaires) (0 réactions) (0 assignés)JavaScript (14 268 stars) (520 forks)batch import
Instant Renderenhancementhelp wantedlevel 1 (super easy)priority: lowserializer

Description

This

(function () {
  let n = global.__abstract ? __abstract("number", "23") : 23;
  if (n === 42) {
    let d = Date.now(); // dead pure generator entry
  }
})();

prepacks to

if (23 === 42) {}

which is obviously suboptimal.

During its fixpoint computation, the visitor should not visit the condition until at least one generator entry cannot be omitted.

Guide contributeur

Stack technique
javascript
Domaine
performance
Type d'issue
bug
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 hours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
JavaScript knowledgeCompiler architecture basics
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40
Direction de recherche
Investigate the fixpoint computation and visitor code in Prepack. Look for the relevant visitor logic that handles if statements and generator entries. Determine how to delay visiting the condition until at least one generator entry is confirmed not omitted. The issue suggests a specific fix in the visitor condition traversal.