X87: [ic] Use the CallFunction builtin to invoke accessors.

X87: [ic] Use the CallFunction builtin to invoke accessors.

  port 6df9a22c3f8d6640e867ca659c95e22289b00b40 (r35187)

  original commit message:
  The HandlerCompiler did not properly handle the weird edge case when a
  sloppy mode function was installed as an accessor on one of the value
  wrapper prototypes and then accessed via a load from a primitive value.
  In this case we just passed the primitive value untouched instead of
  properly wrapping it first. The CallFunction builtin properly deals with
  all the funny edge cases, so we use it instead of duplicating almost all
  of the logic here (the performance difference is neglible).

BUG=

Committed: https://crrev.com/96ddf420c56016c5ef463c61d09b721292ed786e
Cr-Commit-Position: refs/heads/master@{#35522}

[zhengxing.li, 2016-04-15 04:07:00]

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

[zhengxing.li, 2016-04-15 04:07:01]

PTAL, thanks!

[Weiliang, 2016-04-15 06:08:44]

lgtm

[zhengxing.li, 2016-04-15 06:37:11]

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

[commit-bot: I haz the power, 2016-04-15 06:37:13]

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

[zhengxing.li, 2016-04-15 08:10:35]

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

[zhengxing.li, 2016-04-15 08:10:45]

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

[commit-bot: I haz the power, 2016-04-15 08:10:47]

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

[Michael Achenbach, 2016-04-15 11:26:12]

FYI: builders are hanging on this CL. Investigating...

[commit-bot: I haz the power, 2016-04-15 11:58:59]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-04-15 11:59:00]

Try jobs failed on following builders:
  v8_linux_chromium_gn_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_chromium_gn_rel/bu...)
  v8_win64_rel_ng on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_win64_rel_ng/builds/5869)

[Michael Achenbach, 2016-04-15 11:59:33]

The CQ bit was checked by machenbach@chromium.org

[commit-bot: I haz the power, 2016-04-15 11:59:39]

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

[commit-bot: I haz the power, 2016-04-15 12:21:17]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-15 12:21:40]

Description was changed from

==========
X87: [ic] Use the CallFunction builtin to invoke accessors.

  port 6df9a22c3f8d6640e867ca659c95e22289b00b40 (r35187)

  original commit message:
  The HandlerCompiler did not properly handle the weird edge case when a
  sloppy mode function was installed as an accessor on one of the value
  wrapper prototypes and then accessed via a load from a primitive value.
  In this case we just passed the primitive value untouched instead of
  properly wrapping it first. The CallFunction builtin properly deals with
  all the funny edge cases, so we use it instead of duplicating almost all
  of the logic here (the performance difference is neglible).

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

to

==========
X87: [ic] Use the CallFunction builtin to invoke accessors.

  port 6df9a22c3f8d6640e867ca659c95e22289b00b40 (r35187)

  original commit message:
  The HandlerCompiler did not properly handle the weird edge case when a
  sloppy mode function was installed as an accessor on one of the value
  wrapper prototypes and then accessed via a load from a primitive value.
  In this case we just passed the primitive value untouched instead of
  properly wrapping it first. The CallFunction builtin properly deals with
  all the funny edge cases, so we use it instead of duplicating almost all
  of the logic here (the performance difference is neglible).

BUG=

Committed: https://crrev.com/96ddf420c56016c5ef463c61d09b721292ed786e
Cr-Commit-Position: refs/heads/master@{#35522}
==========

[commit-bot: I haz the power, 2016-04-15 12:21:41]

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