[Interpreter] Add expression positions to BinaryOps.

[Interpreter] Add expression positions to BinaryOps.

BUG=v8:5723
Committed: https://crrev.com/bfc53f6ed0a7159f7bbf63c6a2b1e35f57b5b7d7
Cr-Commit-Position: refs/heads/master@{#41583}

[rmcilroy, 2016-12-07 19:06:16]

The CQ bit was checked by rmcilroy@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-07 19:06:21]

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

[rmcilroy, 2016-12-07 19:07:04]

The CQ bit was checked by rmcilroy@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-07 19:07:11]

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

[rmcilroy, 2016-12-07 19:09:04]

rmcilroy@chromium.org changed reviewers:
+ mstarzinger@chromium.org

[rmcilroy, 2016-12-07 19:09:05]

This means we can no longer perform the LdaSmi+Add -> AddSmi peephole
optimization as often if both bytecodes have source positions. I'm not sure if
this will hurt us in the real world - I guess the bots will let us know.

Michi, could you PTAL, Clemens FYI.

[commit-bot: I haz the power, 2016-12-07 19:32:01]

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

[commit-bot: I haz the power, 2016-12-07 19:32:02]

Dry run: This issue passed the CQ dry run.

[Michael Starzinger, 2016-12-08 09:13:21]

LGTM. Thanks!

https://codereview.chromium.org/2555263002/diff/20001/test/message/tonumber_s...
File test/message/tonumber_symbol.js (right):

https://codereview.chromium.org/2555263002/diff/20001/test/message/tonumber_s...
test/message/tonumber_symbol.js:1: // Copyright 2016 the V8 project authors. All
rights reserved.
nit: Can we use a dash instead of an underscore in the filename for consistency?

[rmcilroy, 2016-12-08 09:46:13]

https://codereview.chromium.org/2555263002/diff/20001/test/message/tonumber_s...
File test/message/tonumber_symbol.js (right):

https://codereview.chromium.org/2555263002/diff/20001/test/message/tonumber_s...
test/message/tonumber_symbol.js:1: // Copyright 2016 the V8 project authors. All
rights reserved.
On 2016/12/08 09:13:21, Michael Starzinger wrote:
> nit: Can we use a dash instead of an underscore in the filename for
consistency?

Done.

[rmcilroy, 2016-12-08 09:46:28]

The CQ bit was checked by rmcilroy@chromium.org

[rmcilroy, 2016-12-08 09:46:28]

The patchset sent to the CQ was uploaded after l-g-t-m from
mstarzinger@chromium.org
Link to the patchset: https://codereview.chromium.org/2555263002/#ps40001
(title: "Move test file")

[commit-bot: I haz the power, 2016-12-08 09:46:31]

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-12-08 10:11:06]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1481190388194200,
"parent_rev": "b524cd5da631272b633763114a2c55c8ce7e9180", "commit_rev":
"86869d8d5b9098ab960911324896575ae0147edb"}

[commit-bot: I haz the power, 2016-12-08 10:11:10]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-12-08 10:11:25]

Description was changed from

==========
[Interpreter] Add expression positions to BinaryOps.

BUG=v8:5723
==========

to

==========
[Interpreter] Add expression positions to BinaryOps.

BUG=v8:5723

Committed: https://crrev.com/bfc53f6ed0a7159f7bbf63c6a2b1e35f57b5b7d7
Cr-Commit-Position: refs/heads/master@{#41583}
==========

[commit-bot: I haz the power, 2016-12-08 10:11:26]

Patchset 3 (id:??) landed as
https://crrev.com/bfc53f6ed0a7159f7bbf63c6a2b1e35f57b5b7d7
Cr-Commit-Position: refs/heads/master@{#41583}