Fix how Hydrogen traces inlined functions.

Fix how Hydrogen traces inlined functions.

https://crrev.com/c3a6ca68d0646b10885ef7017557eaf463db2e4a changed what kind of information is dumped which broke some properties of the output that IRHydra relies on:

1. Because IRHydra does not have access to the whole scripts it relies on function sources dumps to map IR and deoptimization to the actual source. This means that source positions must be dumped in a way that allows for this mapping and previously we used positions relative to the beginning of the function to allow that. Now we are dumping absolute script positions which means that function sources must be annotated with the start position so that IRHydra (or any other similar tools) could translate absolute script positions into function-relative ones.

2. When dumping inlined functions we should dump which function are we inlining into because position itself is not enough to recover the inlining stack from the output.

BUG=

Committed: https://crrev.com/7371284dcf57c7b13a5398ab4764362b869aa61e
Cr-Commit-Position: refs/heads/master@{#41663}

[Vyacheslav Egorov (Google), 2016-12-12 22:58:50]

vegorov@google.com changed reviewers:
+ bmeurer@chromium.org, tebbi@chromium.org

[Vyacheslav Egorov (Google), 2016-12-12 22:58:50]

PTAL

[Benedikt Meurer, 2016-12-13 06:10:14]

lgtm

[Vyacheslav Egorov (Google), 2016-12-13 06:42:37]

The CQ bit was checked by vegorov@google.com

[commit-bot: I haz the power, 2016-12-13 06:42:45]

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-13 07:30:23]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1481611357618250, "parent_rev":
"e61e2ee8ba5a380337c904e84cb18f46ed1b8917", "commit_rev":
"2e30434295e804c049832309c70935db94a77009"}

[commit-bot: I haz the power, 2016-12-13 07:30:29]

Description was changed from

==========
Fix how Hydrogen traces inlined functions.

https://crrev.com/c3a6ca68d0646b10885ef7017557eaf463db2e4a changed what kind of
information is dumped which broke some properties of the output that IRHydra
relies on:

1. Because IRHydra does not have access to the whole scripts it relies on
function sources dumps to map IR and deoptimization to the actual source. This
means that source positions must be dumped in a way that allows for this mapping
and previously we used positions relative to the beginning of the function to
allow that. Now we are dumping absolute script positions which means that
function sources must be annotated with the start position so that IRHydra (or
any other similar tools) could translate absolute script positions into
function-relative ones.

2. When dumping inlined functions we should dump which function are we inlining
into because position itself is not enough to recover the inlining stack from
the output.

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

to

==========
Fix how Hydrogen traces inlined functions.

https://crrev.com/c3a6ca68d0646b10885ef7017557eaf463db2e4a changed what kind of
information is dumped which broke some properties of the output that IRHydra
relies on:

1. Because IRHydra does not have access to the whole scripts it relies on
function sources dumps to map IR and deoptimization to the actual source. This
means that source positions must be dumped in a way that allows for this mapping
and previously we used positions relative to the beginning of the function to
allow that. Now we are dumping absolute script positions which means that
function sources must be annotated with the start position so that IRHydra (or
any other similar tools) could translate absolute script positions into
function-relative ones.

2. When dumping inlined functions we should dump which function are we inlining
into because position itself is not enough to recover the inlining stack from
the output.

BUG=

Review-Url: https://codereview.chromium.org/2573653002
==========

[commit-bot: I haz the power, 2016-12-13 07:30:30]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-13 07:31:40]

Description was changed from

==========
Fix how Hydrogen traces inlined functions.

https://crrev.com/c3a6ca68d0646b10885ef7017557eaf463db2e4a changed what kind of
information is dumped which broke some properties of the output that IRHydra
relies on:

1. Because IRHydra does not have access to the whole scripts it relies on
function sources dumps to map IR and deoptimization to the actual source. This
means that source positions must be dumped in a way that allows for this mapping
and previously we used positions relative to the beginning of the function to
allow that. Now we are dumping absolute script positions which means that
function sources must be annotated with the start position so that IRHydra (or
any other similar tools) could translate absolute script positions into
function-relative ones.

2. When dumping inlined functions we should dump which function are we inlining
into because position itself is not enough to recover the inlining stack from
the output.

BUG=

Review-Url: https://codereview.chromium.org/2573653002
==========

to

==========
Fix how Hydrogen traces inlined functions.

https://crrev.com/c3a6ca68d0646b10885ef7017557eaf463db2e4a changed what kind of
information is dumped which broke some properties of the output that IRHydra
relies on:

1. Because IRHydra does not have access to the whole scripts it relies on
function sources dumps to map IR and deoptimization to the actual source. This
means that source positions must be dumped in a way that allows for this mapping
and previously we used positions relative to the beginning of the function to
allow that. Now we are dumping absolute script positions which means that
function sources must be annotated with the start position so that IRHydra (or
any other similar tools) could translate absolute script positions into
function-relative ones.

2. When dumping inlined functions we should dump which function are we inlining
into because position itself is not enough to recover the inlining stack from
the output.

BUG=

Committed: https://crrev.com/7371284dcf57c7b13a5398ab4764362b869aa61e
Cr-Commit-Position: refs/heads/master@{#41663}
==========

[commit-bot: I haz the power, 2016-12-13 07:31:41]

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