facebookarchive/prepack

Fuse adjacent JOIN_GENERATORS with same path condition

Open

#2565 aperta il 22 set 2018

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)JavaScript (520 fork)batch import
Instant Renderenhancementhelp wantedlevel 2 (easy)

Metriche repository

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

Descrizione

This...

let x = __abstract("boolean", "(x)");
if (x) console.log("Hello");
if (x) console.log(" World");

currently prepacks to

(function () {
  var _0 = x;

  if (_0) {
    console.log("Hello");
  }

  if (_0) {
    console.log(" World");
  }
})();

but should Prepack to

(function () {
  var _0 = x;

  if (_0) {
    console.log("Hello");
    console.log(" World");
  }
})();

(Seen arising from actual product code.)

Guida contributor

Direzione di ricerca
Esamina il codice sorgente di Prepack, probabilmente nelle directory src/ o lib/, per capire come vengono gestiti i JOIN GENERATORS. Cerca il pattern di ottimizzazione specifico in cui JOIN GENERATORS adiacenti con la stessa condizione vengono fusi. Il problema mostra una chiara trasformazione prima/dopo, quindi la correzione implica modificare la logica che emette questi generatori per unire quelli consecutivi quando la condizione è identica. Esamina i test esistenti per ottimizzazioni di fusione simili per garantire la correttezza. Nessun PR o commento collegato fornisce ulteriori indicazioni.
Tech stack
javascript
Dominio
performance
Tipo issue
Performance
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
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
basic understanding of JavaScriptfamiliarity with Prepack internals
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60