chakra-core/ChakraCore

Optimize OP_LdCustomSpreadIteratorList for arguments object

Open

#1.230 geöffnet am 1. Juli 2016

Auf GitHub ansehen
 (10 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (1.374 Forks)batch import
PerformanceProposalSuggestiongood first issuehelp wanted

Repository-Metriken

Stars
 (9.000 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

I was looking into the code of OP_LdCustomSpreadIteratorList and noticed that we optimize it by not creating SpreadArgument if parameter passed to OP_LdCustomSpreadIteratorList does not have user defined iterator. We do this check only for JavascriptArray and TypedArrayBase and we can expand it for ArgumentsObject as well. As per spec, both unmapped and mapped argument objects has default @@iterator set to Array.proto.values.

Contributor Guide

Research-Richtung
Untersuche die vorhandene Optimierung in lib/Runtime/Language/JavascriptOperators.cpp in Zeile 1405. Verstehe, wie die Prüfung auf nicht benutzerdefinierte Iteratoren für JavascriptArray und TypedArrayBase durchgeführt wird. Erweitere dann dieselbe Logik auf ArgumentsObject, um sicherzustellen, dass der Standarditerator des Arguments Objekts (Array.prototype.values) verwendet wird. Berücksichtige Randfälle, in denen das Arguments Objekt seinen @@iterator überschrieben haben könnte. Schreibe Tests, um die Optimierung abzudecken.
Tech Stack
cppjavascript
Domain
backend
Issue Type
Performance
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C++JavaScript
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
25