[modules] Fix bugs in assignments to exported variables.

[modules] Fix bugs in assignments to exported variables.

- Add hole check if needed.
- Preserve the accumulator so that the result is the rhs.

R=adamk@chromium.org
BUG=v8:1569, v8:5547
Committed: https://crrev.com/cadfe092a25fd052294a45a2c5aabe34b475f534
Cr-Commit-Position: refs/heads/master@{#40510}

[adamk, 2016-10-20 17:30:16]

lgtm (though we'll need an interpreter OWNER)

Please mention the return value fix in the CL description.

[neis, 2016-10-21 10:03:50]

Description was changed from

==========
[modules] Add missing hole check when assigning to exported variable.

R=adamk@chromium.org
BUG=v8:1569,v8:5547
==========

to

==========
[modules] Fix bugs in assignments to exported variables.

- Add hole check if needed.
- Preserve the accumulator so that the result is the rhs.

R=adamk@chromium.org
BUG=v8:1569,v8:5547
==========

[neis, 2016-10-21 10:03:50]

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

[neis, 2016-10-21 10:04:16]

On 2016/10/20 17:30:16, adamk wrote:
> lgtm (though we'll need an interpreter OWNER)
> 
> Please mention the return value fix in the CL description.

Done.

[adamk, 2016-10-21 13:28:30]

adamk@chromium.org changed reviewers:
+ mythria@chromium.org
- rmcilroy@chromium.org

[adamk, 2016-10-21 13:28:31]

-ross, +mythri

[mythria, 2016-10-21 13:51:30]

lgtm.

[neis, 2016-10-21 13:52:31]

The CQ bit was checked by neis@chromium.org

[neis, 2016-10-21 13:52:32]

The patchset sent to the CQ was uploaded after l-g-t-m from adamk@chromium.org
Link to the patchset: https://codereview.chromium.org/2438653003/#ps20001
(title: "Rebase.")

[commit-bot: I haz the power, 2016-10-21 13:52:34]

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-10-21 14:24:04]

Description was changed from

==========
[modules] Fix bugs in assignments to exported variables.

- Add hole check if needed.
- Preserve the accumulator so that the result is the rhs.

R=adamk@chromium.org
BUG=v8:1569,v8:5547
==========

to

==========
[modules] Fix bugs in assignments to exported variables.

- Add hole check if needed.
- Preserve the accumulator so that the result is the rhs.

R=adamk@chromium.org
BUG=v8:1569,v8:5547
==========

[commit-bot: I haz the power, 2016-10-21 14:24:05]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-17 22:10:04]

Description was changed from

==========
[modules] Fix bugs in assignments to exported variables.

- Add hole check if needed.
- Preserve the accumulator so that the result is the rhs.

R=adamk@chromium.org
BUG=v8:1569,v8:5547
==========

to

==========
[modules] Fix bugs in assignments to exported variables.

- Add hole check if needed.
- Preserve the accumulator so that the result is the rhs.

R=adamk@chromium.org
BUG=v8:1569,v8:5547

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

[commit-bot: I haz the power, 2016-11-17 22:10:05]

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