[turbofan] Fix usage of ThisFunction parameter in OSR.

[turbofan] Fix usage of ThisFunction parameter in OSR.

R=mstarzinger@chromium.org
BUG=

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

[Michael Starzinger, 2015-01-30 10:06:02]

LGTM.

https://codereview.chromium.org/892593002/diff/1/src/compiler/linkage-impl.h
File src/compiler/linkage-impl.h (right):

https://codereview.chromium.org/892593002/diff/1/src/compiler/linkage-impl.h#...
src/compiler/linkage-impl.h:63: is_osr ? stackloc(-1 - js_parameter_count)
Magic "-1" is being very magic. Is there any constant in frames.h that we can
DCHECK here to make sure this "-1" is accurate?

[titzer, 2015-01-30 12:18:18]

On 2015/01/30 10:06:02, Michael Starzinger wrote:
> LGTM.
> 
> https://codereview.chromium.org/892593002/diff/1/src/compiler/linkage-impl.h
> File src/compiler/linkage-impl.h (right):
> 
>
https://codereview.chromium.org/892593002/diff/1/src/compiler/linkage-impl.h#...
> src/compiler/linkage-impl.h:63: is_osr ? stackloc(-1 - js_parameter_count)
> Magic "-1" is being very magic. Is there any constant in frames.h that we can
> DCHECK here to make sure this "-1" is accurate?

Done. Put a constant in linkage.h.

[titzer, 2015-01-30 12:18:57]

The CQ bit was checked by titzer@chromium.org

[commit-bot: I haz the power, 2015-01-30 12:19:26]

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

[commit-bot: I haz the power, 2015-01-30 13:09:40]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-01-30 13:09:52]

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