X87: Rework CallApi*Stubs.

X87: Rework CallApi*Stubs.

  port 5096492f1b0460b2644c7f79f9ccc5881fc65c50 (r34627)

  original commit message:
  - Eliminate stubs with a variable number of arguments.
    (That only worked due to their very limited use. These
     stubs' interface descriptors were basically lying
     about their number of args, which will fail when used
     generically.)
  - Fix all CallApi*Stubs' interface descriptors to no
    longer lie about their arguments.
  - Unify CallApi*Stub, for * in Function, Accessor,
    FunctionWithFixedArgs.
    (Since these are now all doing the same thing.)
  - Rename the unified stub (and interface descriptors) to
    *ApiCallback*, since that's really what they're doing.
  - Refuse inlining an API callback if its number of
    parameters exceeds the supported number of args.

BUG=

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

[zhengxing.li, 2016-03-10 02:50:04]

zhengxing.li@intel.com changed reviewers:
+ chunyang.dai@intel.com, vogelheim@chromium.org, weiliang.lin@intel.com

[zhengxing.li, 2016-03-10 02:50:05]

PTAL, thanks!

[Weiliang, 2016-03-10 03:53:25]

lgtm

[Weiliang, 2016-03-10 03:53:26]

lgtm

lgtm

[zhengxing.li, 2016-03-10 06:15:24]

The CQ bit was checked by zhengxing.li@intel.com

[commit-bot: I haz the power, 2016-03-10 06:15:36]

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

[commit-bot: I haz the power, 2016-03-10 06:30:57]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-10 06:31:06]

Description was changed from

==========
X87: Rework CallApi*Stubs.

  port 5096492f1b0460b2644c7f79f9ccc5881fc65c50 (r34627)

  original commit message:
  - Eliminate stubs with a variable number of arguments.
    (That only worked due to their very limited use. These
     stubs' interface descriptors were basically lying
     about their number of args, which will fail when used
     generically.)
  - Fix all CallApi*Stubs' interface descriptors to no
    longer lie about their arguments.
  - Unify CallApi*Stub, for * in Function, Accessor,
    FunctionWithFixedArgs.
    (Since these are now all doing the same thing.)
  - Rename the unified stub (and interface descriptors) to
    *ApiCallback*, since that's really what they're doing.
  - Refuse inlining an API callback if its number of
    parameters exceeds the supported number of args.

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

to

==========
X87: Rework CallApi*Stubs.

  port 5096492f1b0460b2644c7f79f9ccc5881fc65c50 (r34627)

  original commit message:
  - Eliminate stubs with a variable number of arguments.
    (That only worked due to their very limited use. These
     stubs' interface descriptors were basically lying
     about their number of args, which will fail when used
     generically.)
  - Fix all CallApi*Stubs' interface descriptors to no
    longer lie about their arguments.
  - Unify CallApi*Stub, for * in Function, Accessor,
    FunctionWithFixedArgs.
    (Since these are now all doing the same thing.)
  - Rename the unified stub (and interface descriptors) to
    *ApiCallback*, since that's really what they're doing.
  - Refuse inlining an API callback if its number of
    parameters exceeds the supported number of args.

BUG=

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

[commit-bot: I haz the power, 2016-03-10 06:31:07]

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