chakra-core/ChakraCore

Optimize OP_LdCustomSpreadIteratorList for arguments object

Open

#1,230 opened on 2016年7月1日

GitHub で見る
 (10 comments) (0 reactions) (0 assignees)JavaScript (1,374 forks)batch import
PerformanceProposalSuggestiongood first issuehelp wanted

Repository metrics

Stars
 (9,000 stars)
PR merge metrics
 (30d に merged PR はありません)

説明

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.

コントリビューターガイド

調査方針
lib/Runtime/Language/JavascriptOperators.cpp の 1405 行目にある既存の最適化を調査します。JavascriptArray と TypedArrayBase に対して、非ユーザー定義イテレータのチェックがどのように行われているかを理解します。その後、同じロジックを ArgumentsObject に拡張し、arguments オブジェクトのデフォルトイテレータ（Array.prototype.values）が使用されるようにします。arguments オブジェクトの @@iterator が上書きされる可能性があるエッジケースを考慮します。この最適化をカバーするテストを作成します。
技術スタック
cppjavascript
領域
backend
Issue 種別
パフォーマンス
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
3
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-2日
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
アクティブ
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
明確
前提条件
C++JavaScript
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
25