[interpreter] Add CreateCatchContext bytecode

[interpreter] Add CreateCatchContext bytecode

One more bytecode to pass info through to TurboFan.

BUG=v8:4280
LOG=n

Committed: https://crrev.com/188ea8c5410bec100723bbf231c6b99ec50e41cc
Cr-Commit-Position: refs/heads/master@{#38726}

[klaasb, 2016-08-18 13:56:03]

The CQ bit was checked by klaasb@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-08-18 13:56:05]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[klaasb, 2016-08-18 14:02:18]

klaasb@google.com changed reviewers:
+ mstarzinger@chromium.org, rmcilroy@chromium.org

[commit-bot: I haz the power, 2016-08-18 14:09:04]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-18 14:09:05]

Dry run: Try jobs failed on following builders:
  v8_linux64_gyp_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_gyp_rel_ng/build...)
  v8_linux64_gyp_rel_ng_triggered on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_gyp_rel_ng_trigg...)

[klaasb, 2016-08-18 14:13:51]

The CQ bit was checked by klaasb@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-08-18 14:14:02]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-18 14:16:14]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-18 14:16:15]

Dry run: Try jobs failed on following builders:
  v8_mac_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_mac_rel_ng/builds/7194)

[klaasb, 2016-08-18 14:27:39]

The CQ bit was checked by klaasb@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-08-18 14:27:44]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Michael Starzinger, 2016-08-18 15:00:43]

LGTM from my end.

https://codereview.chromium.org/2260473003/diff/40001/src/interpreter/interpr...
File src/interpreter/interpreter.cc (right):

https://codereview.chromium.org/2260473003/diff/40001/src/interpreter/interpr...
src/interpreter/interpreter.cc:1827: // the variable name at |index| for and the
closure in the accumulator.
nit: Stray "for" in comment.

[klaasb, 2016-08-18 15:21:42]

https://codereview.chromium.org/2260473003/diff/40001/src/interpreter/interpr...
File src/interpreter/interpreter.cc (right):

https://codereview.chromium.org/2260473003/diff/40001/src/interpreter/interpr...
src/interpreter/interpreter.cc:1827: // the variable name at |index| for and the
closure in the accumulator.
On 2016/08/18 15:00:42, Michael Starzinger wrote:
> nit: Stray "for" in comment.

Done.

[rmcilroy, 2016-08-18 15:39:11]

LGTM with a nit

https://codereview.chromium.org/2260473003/diff/60001/test/unittests/interpre...
File test/unittests/interpreter/bytecode-array-builder-unittest.cc (right):

https://codereview.chromium.org/2260473003/diff/60001/test/unittests/interpre...
test/unittests/interpreter/bytecode-array-builder-unittest.cc:107: // Emit
create context operation.
nit - move comment to operation above, and group the context creation calls
(including CreateWithContext)

[klaasb, 2016-08-18 16:25:40]

https://codereview.chromium.org/2260473003/diff/60001/test/unittests/interpre...
File test/unittests/interpreter/bytecode-array-builder-unittest.cc (right):

https://codereview.chromium.org/2260473003/diff/60001/test/unittests/interpre...
test/unittests/interpreter/bytecode-array-builder-unittest.cc:107: // Emit
create context operation.
On 2016/08/18 15:39:11, rmcilroy wrote:
> nit - move comment to operation above, and group the context creation calls
> (including CreateWithContext)

Done.

[klaasb, 2016-08-18 16:25:46]

The CQ bit was checked by klaasb@google.com

[klaasb, 2016-08-18 16:25:46]

The patchset sent to the CQ was uploaded after l-g-t-m from
mstarzinger@chromium.org, rmcilroy@chromium.org
Link to the patchset: https://codereview.chromium.org/2260473003/#ps80001
(title: "group context bytecodes")

[commit-bot: I haz the power, 2016-08-18 16:25:52]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-18 16:51:21]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-08-18 16:51:33]

Description was changed from

==========
[interpreter] Add CreateCatchContext bytecode

One more bytecode to pass info through to TurboFan.

BUG=v8:4280
LOG=n
==========

to

==========
[interpreter] Add CreateCatchContext bytecode

One more bytecode to pass info through to TurboFan.

BUG=v8:4280
LOG=n

Committed: https://crrev.com/188ea8c5410bec100723bbf231c6b99ec50e41cc
Cr-Commit-Position: refs/heads/master@{#38726}
==========

[commit-bot: I haz the power, 2016-08-18 16:51:34]

Patchset 5 (id:??) landed as
https://crrev.com/188ea8c5410bec100723bbf231c6b99ec50e41cc
Cr-Commit-Position: refs/heads/master@{#38726}