chakra-core/ChakraCore

Optimize OP_LdCustomSpreadIteratorList for arguments object

Open

#1.230 aberto em 1 de jul. de 2016

Ver no GitHub
 (10 comments) (0 reactions) (0 assignees)JavaScript (1.374 forks)batch import
PerformanceProposalSuggestiongood first issuehelp wanted

Métricas do repositório

Stars
 (9.000 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

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.

Guia do colaborador

Direção de pesquisa
Examine a otimização existente em lib/Runtime/Language/JavascriptOperators.cpp na linha 1405. Entenda como a verificação para iterador não definido pelo usuário é feita para JavascriptArray e TypedArrayBase. Em seguida, estenda a mesma lógica para ArgumentsObject, garantindo que o iterador padrão do objeto arguments (Array.prototype.values) seja usado. Considere casos extremos em que o objeto arguments pode ter seu @@iterator sobrescrito. Escreva testes para cobrir a otimização.
Pilha de tecnologia
cppjavascript
Domain
backend
Tipo Issue
Performance
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-2 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C++JavaScript
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
25