CpuProfiler: x87. put right address to the stack, so the callee would be able to resolve it into th…

CpuProfiler: x87. put right address to the stack, so the callee would be able to resolve it into the right deopt_info.

'from' is using for Code object lookup and will be used for
inline_id lookup. see https://codereview.chromium.org/1012633002
So we should be able to map it.

BUG=chromium:452067
LOG=n

Committed: https://crrev.com/549d5dde6a454343fe0edd4b52327724265e0dc2
Cr-Commit-Position: refs/heads/master@{#27251}

[loislo, 2015-03-17 17:38:23]

loislo@chromium.org changed reviewers:
+ alph@chromium.org, jarin@chromium.org, svenpanne@chromium.org

[loislo, 2015-03-17 17:38:24]

PTAL

[alph, 2015-03-17 18:01:58]

answer the comment and then lgtm

https://codereview.chromium.org/1013243002/diff/1/src/x87/lithium-codegen-x87.cc
File src/x87/lithium-codegen-x87.cc (right):

https://codereview.chromium.org/1013243002/diff/1/src/x87/lithium-codegen-x87...
src/x87/lithium-codegen-x87.cc:392: if (info()->saves_caller_doubles())
RestoreCallerDoubles();
why's this?

[loislo, 2015-03-17 18:05:18]

On 2015/03/17 at 18:01:58, alph wrote:
> answer the comment and then lgtm
> 
>
https://codereview.chromium.org/1013243002/diff/1/src/x87/lithium-codegen-x87.cc
> File src/x87/lithium-codegen-x87.cc (right):
> 
>
https://codereview.chromium.org/1013243002/diff/1/src/x87/lithium-codegen-x87...
> src/x87/lithium-codegen-x87.cc:392: if (info()->saves_caller_doubles())
RestoreCallerDoubles();
> why's this?

fixed

[Sven Panne, 2015-03-18 08:04:24]

lgtm

[loislo, 2015-03-18 08:06:54]

The CQ bit was checked by loislo@chromium.org

[loislo, 2015-03-18 08:06:54]

The patchset sent to the CQ was uploaded after l-g-t-m from alph@chromium.org
Link to the patchset: https://codereview.chromium.org/1013243002/#ps20001
(title: "comments were added")

[commit-bot: I haz the power, 2015-03-18 08:07:10]

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

[commit-bot: I haz the power, 2015-03-18 08:08:33]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-03-18 08:08:52]

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