X87: [calls] Consistent call protocol for calls.

X87: [calls] Consistent call protocol for calls.

port b37907ff7f866873ddfbfc97670b43c19a5fc7f9 (r30648).

original commit message:

    The number of actual arguments should always be available, there's no
    point in trying to optimize away a simple assignment of an immediate to
    a register before some calls.

    The main motivation is to have a consistent state at the beginning of every
    function. Currently the arguments register (i.e. rax or eax) either contains
    the number of arguments or some random garbage depending on whether
    the callsite decided that the callee might need the information or not.
    This causes trouble with runtime implementations of functions that
    do not set internal_formal_parameter_count to the DontAdaptArguments
    sentinel (we don't have any of those yet), but also makes it impossible
    to sanity check the arguments in the callee, because the callee doesn't
    know whether the caller decided to pass the number of arguments or
    random garbage.

BUG=

Committed: https://crrev.com/99f0130782a57a9abcaea7da5f2a495fe3cb2071
Cr-Commit-Position: refs/heads/master@{#30669}

[chunyang.dai, 2015-09-10 03:40:52]

chunyang.dai@intel.com changed reviewers:
+ weiliang.lin@intel.com

[chunyang.dai, 2015-09-10 03:40:52]

PTAL

[Weiliang, 2015-09-10 05:14:17]

The CQ bit was checked by weiliang.lin@intel.com

[Weiliang, 2015-09-10 05:14:17]

lgtm

[commit-bot: I haz the power, 2015-09-10 05:14:22]

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

[commit-bot: I haz the power, 2015-09-10 05:42:45]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-09-10 05:43:02]

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