[turbofan] Jump threading awareness of frame elision.

Preparing the terrain for frame elision. This change is necessary to
avoid jump threading erasing the reconstruction of a frame, if the
frame was elided.

BUG=

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

[Mircea Trofin, 2016-01-28 03:27:03]

Description was changed from

==========
Jump threading awareness of frame elision.

BUG=
==========

to

==========
Preparing the terrain for frame elision. This change is necessary to
avoid jump threading erasing the reconstruction of a frame, if the
frame was elided.

BUG=
==========

[Mircea Trofin, 2016-01-28 03:27:03]

mtrofin@chromium.org changed reviewers:
+ bmeurer@chromium.org, jarin@chromium.org

[Jarin, 2016-01-28 05:26:59]

lgtm (assuming the monster condition is moved from the pipeline to the register
allocator).

https://codereview.chromium.org/1642823002/diff/1/src/compiler/jump-threading.h
File src/compiler/jump-threading.h (right):

https://codereview.chromium.org/1642823002/diff/1/src/compiler/jump-threading...
src/compiler/jump-threading.h:21: InstructionSequence* code, bool
frame_at_start);
Please explain what the {frame_at_start} parameter means in the comment above.

https://codereview.chromium.org/1642823002/diff/1/src/compiler/pipeline.cc
File src/compiler/pipeline.cc (right):

https://codereview.chromium.org/1642823002/diff/1/src/compiler/pipeline.cc#ne...
src/compiler/pipeline.cc:1338: // TODO(mtrofin): move this off to the register
allocator.
Yes, please do.

[Mircea Trofin, 2016-01-28 05:36:18]

On 2016/01/28 05:26:59, Jarin wrote:
> lgtm (assuming the monster condition is moved from the pipeline to the
register
> allocator).
> 
>
https://codereview.chromium.org/1642823002/diff/1/src/compiler/jump-threading.h
> File src/compiler/jump-threading.h (right):
> 
>
https://codereview.chromium.org/1642823002/diff/1/src/compiler/jump-threading...
> src/compiler/jump-threading.h:21: InstructionSequence* code, bool
> frame_at_start);
> Please explain what the {frame_at_start} parameter means in the comment above.
> 
> https://codereview.chromium.org/1642823002/diff/1/src/compiler/pipeline.cc
> File src/compiler/pipeline.cc (right):
> 
>
https://codereview.chromium.org/1642823002/diff/1/src/compiler/pipeline.cc#ne...
> src/compiler/pipeline.cc:1338: // TODO(mtrofin): move this off to the register
> allocator.
> Yes, please do.

Are we OK with waiting until frame elision makes it in before moving the monster
condition?

[Mircea Trofin, 2016-01-28 16:11:17]

The CQ bit was checked by mtrofin@chromium.org

[commit-bot: I haz the power, 2016-01-28 16:11:27]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1642823002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1642823002/1

[commit-bot: I haz the power, 2016-01-28 16:14:44]

Description was changed from

==========
Preparing the terrain for frame elision. This change is necessary to
avoid jump threading erasing the reconstruction of a frame, if the
frame was elided.

BUG=
==========

to

==========
Preparing the terrain for frame elision. This change is necessary to
avoid jump threading erasing the reconstruction of a frame, if the
frame was elided.

BUG=
==========

[commit-bot: I haz the power, 2016-01-28 16:14:45]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-01-28 16:15:21]

Description was changed from

==========
Preparing the terrain for frame elision. This change is necessary to
avoid jump threading erasing the reconstruction of a frame, if the
frame was elided.

BUG=
==========

to

==========
Preparing the terrain for frame elision. This change is necessary to
avoid jump threading erasing the reconstruction of a frame, if the
frame was elided.

BUG=

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

[commit-bot: I haz the power, 2016-01-28 16:15:21]

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