[turbofan] Preserve pending message while inside finally-block.

[turbofan] Preserve pending message while inside finally-block.

This makes sure that any pending message is saved before entering
and restored after exiting a finally block. It also makes sure that
operand stacks are kept in sync to full-codegen.

R=bmeurer@chromium.org
TEST=cctest/test-run-jsexceptions/ThrowMessage

Committed: https://crrev.com/e6ff16d6bfb87fe21ffbf9120afa2be703e3327d
Cr-Commit-Position: refs/heads/master@{#27036}

[Benedikt Meurer, 2015-03-06 04:50:49]

https://codereview.chromium.org/979173002/diff/40001/src/compiler/ast-graph-b...
File src/compiler/ast-graph-builder.cc (right):

https://codereview.chromium.org/979173002/diff/40001/src/compiler/ast-graph-b...
src/compiler/ast-graph-builder.cc:2825: jsgraph()->ZeroConstant());
This is probably wrong for the generic pipeline and only works because Smi 0 and
intptr_t 0 is currently identical. You should use IntPtrConstant(0) instead.

https://codereview.chromium.org/979173002/diff/40001/src/compiler/ast-graph-b...
src/compiler/ast-graph-builder.cc:2834: jsgraph()->ZeroConstant(), value);
Same here.

[Michael Starzinger, 2015-03-06 09:47:36]

https://codereview.chromium.org/979173002/diff/40001/src/compiler/ast-graph-b...
File src/compiler/ast-graph-builder.cc (right):

https://codereview.chromium.org/979173002/diff/40001/src/compiler/ast-graph-b...
src/compiler/ast-graph-builder.cc:2825: jsgraph()->ZeroConstant());
On 2015/03/06 04:50:49, Benedikt Meurer wrote:
> This is probably wrong for the generic pipeline and only works because Smi 0
and
> intptr_t 0 is currently identical. You should use IntPtrConstant(0) instead.

Done.

https://codereview.chromium.org/979173002/diff/40001/src/compiler/ast-graph-b...
src/compiler/ast-graph-builder.cc:2834: jsgraph()->ZeroConstant(), value);
On 2015/03/06 04:50:49, Benedikt Meurer wrote:
> Same here.

Done.

[Benedikt Meurer, 2015-03-06 09:48:03]

Nice, thanks. LGTM

[Michael Starzinger, 2015-03-06 10:08:32]

The CQ bit was checked by mstarzinger@chromium.org

[commit-bot: I haz the power, 2015-03-06 10:08:46]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/979173002/80001

[commit-bot: I haz the power, 2015-03-06 10:15:54]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2015-03-06 10:16:11]

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