chakra-core/ChakraCore

Optimize OP_LdCustomSpreadIteratorList for arguments object

Open

#1230 aperta il 1 lug 2016

Vedi su GitHub
 (10 commenti) (0 reazioni) (0 assegnatari)JavaScript (1374 fork)batch import
PerformanceProposalSuggestiongood first issuehelp wanted

Metriche repository

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

Descrizione

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.

Guida contributor

Direzione di ricerca
Esamina l'ottimizzazione esistente in lib/Runtime/Language/JavascriptOperators.cpp alla riga 1405. Comprendi come viene effettuato il controllo per l'iteratore non definito dall'utente per JavascriptArray e TypedArrayBase. Quindi estendi la stessa logica a ArgumentsObject, assicurandoti che venga utilizzato l'iteratore predefinito dell'oggetto arguments (Array.prototype.values). Considera i casi limite in cui l'oggetto arguments potrebbe avere il suo @@iterator sovrascritto. Scrivi test per coprire l'ottimizzazione.
Tech stack
cppjavascript
Dominio
backend
Tipo issue
Performance
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++JavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
25