google/closure-compiler

Array constructor with length is not eliminated

Open

#4046 aperta il 22 gen 2023

Vedi su GitHub
 (12 commenti) (0 reazioni) (0 assegnatari)Java (1197 fork)batch import
P4featgood first issuehelp wanted

Metriche repository

Star
 (7176 star)
Metriche merge PR
 (Merge medio 12g 15h) (1 PR mergiata in 30 g)

Descrizione

Consider the code:

/** @const {!Array<number>} */
const a = [1];

/** @const {!Array<number>} */
const b = Array(10);
b[0] = 2;

/** @const {!Array<number>} */
const c = [];
c.push(3);

/** @const {!Array<number>} */
const d = Array(10);
d.push(4);

When compiled with

yarn google-closure-compiler -O ADVANCED --js a.js --rewrite_polyfills=false

we get

Array(10)[0]=2;Array(10).push(4);

Arrays created with the Array(len) constructor are not eliminated.

google-closure-compiler version: v20230103

Guida contributor

Direzione di ricerca
Indaga sul passaggio di eliminazione del codice morto di closure compiler per gli array creati con il costruttore Array(length). Esamina il codice sorgente nel modulo di ottimizzazione del compilatore, in particolare le parti che gestiscono i letterali di array e le chiamate al costruttore. Il problema mostra che Array(10) non viene rimosso mentre i letterali di array come [1] e [] sì. La correzione probabilmente comporta l'aggiunta di un modello per le chiamate al costruttore Array nella logica di eliminazione del codice morto. Controlla i commenti per eventuali suggerimenti dei manutentori e considera di testare compilando il compilatore ed eseguendo l'esempio fornito.
Tech stack
javascriptjava
Dominio
performancebuild system
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
4
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.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
Understanding of JavaScriptFamiliarity with closure compiler internals
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
30