google/closure-compiler

Array constructor with length is not eliminated

Open

#4.046 geöffnet am 22. Jan. 2023

Auf GitHub ansehen
 (12 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Java (7.176 Stars) (1.197 Forks)batch import
P4featgood first issuehelp wanted

Beschreibung

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

Contributor Guide

Tech Stack
javascriptjava
Domain
performancebuild system
Issue Type
bug
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
4
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 hours
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
active
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
mostly clear
Voraussetzungen
Understanding of JavaScriptFamiliarity with closure compiler internals
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
30
Research-Richtung
Investigate the closure compiler's dead code elimination pass for arrays created with the Array(length) constructor. Look at the source code in the compiler's optimization module, particularly the parts that deal with array literals and constructor calls. The issue shows that Array(10) is not being removed while array literals like [1] and [] are. The fix likely involves adding a pattern for Array constructor calls in the dead code elimination logic. Check the comments for any maintainer hints and consider testing by building the compiler and running the provided example.