facebookarchive/prepack

Prepack emits if-statements with no consequent or alternate code

Open

#2220 aperta il 6 lug 2018

Vedi su GitHub
 (3 commenti) (0 reazioni) (0 assegnatari)JavaScript (520 fork)batch import
Instant Renderenhancementhelp wantedlevel 1 (super easy)priority: lowserializer

Metriche repository

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

Descrizione

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.

Guida contributor

Direzione di ricerca
Indaga sul calcolo del punto fisso e sul codice del visitor in Prepack. Cerca la logica del visitor pertinente che gestisce le istruzioni if e le voci del generatore. Determina come ritardare la visita della condizione fino a quando almeno una voce del generatore non sia confermata non omessa. Il problema suggerisce una correzione specifica nella traversata della condizione del visitor.
Tech stack
javascript
Dominio
performance
Tipo issue
Bug
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
JavaScript knowledgeCompiler architecture basics
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40