X87: Revert of [builtins] Introduce a builtin for Abort(). (patchset #5 id:80001 of https://coderev…

X87: Revert of [builtins] Introduce a builtin for Abort(). (patchset #5 id:80001 of https://codereview.chromium.org/2156923002/ ).

  port 3e8f49ab59206b587c6c44a58eda954a76355409 (r37883)

  original commit message:
  Reason for revert:
  Blocks roll: https://build.chromium.org/p/client.v8.ports/builders/V8%20Linux%20-%20arm64%20-%20sim%20-%20debug/builds/1622

  Original issue's description:
  > [builtins] Introduce a builtin for Abort().
  >
  > Calling Runtime::kAbort through a builtin instead of the c-entry stub
  > will allow to generate the call in a background thread, because a
  > builtin provides its own handle, whereas a code stub does not.
  >
  > @v8-mips-ports: Could you take a special look at the padding that is
  > done in MacroAssembler::Abort()?
  >
  > R=bmeurer@chromium.org, titzer@chromium.org, mstarzinger@chromium.org, v8-mips-ports@googlegroups.com, v8-arm-ports@googlegroups.com
  >
  > Committed: https://crrev.com/9be015a254cfff871c56cd129523a729637e9158
  > Cr-Commit-Position: refs/heads/master@{#37854}

BUG=

Committed: https://crrev.com/870fac08c921d0f6bb42dd8b5344296a478073e8
Cr-Commit-Position: refs/heads/master@{#37888}

[zhengxing.li, 2016-07-20 08:03:01]

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

[zhengxing.li, 2016-07-20 08:03:01]

PTAL, thanks!

[Michael Hablich, 2016-07-20 08:07:56]

On 2016/07/20 08:03:01, zhengxing.li wrote:
> PTAL, thanks!

lgtm

[Michael Hablich, 2016-07-20 08:08:00]

The CQ bit was checked by hablich@chromium.org

[commit-bot: I haz the power, 2016-07-20 08:08:10]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Weiliang, 2016-07-20 08:08:45]

lgtm

[commit-bot: I haz the power, 2016-07-20 08:32:17]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-20 08:32:19]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-07-20 08:34:54]

Description was changed from

==========
X87: Revert of [builtins] Introduce a builtin for Abort(). (patchset #5 id:80001
of https://codereview.chromium.org/2156923002/ ).

  port 3e8f49ab59206b587c6c44a58eda954a76355409 (r37883)

  original commit message:
  Reason for revert:
  Blocks roll:
https://build.chromium.org/p/client.v8.ports/builders/V8%20Linux%20-%20arm64%...

  Original issue's description:
  > [builtins] Introduce a builtin for Abort().
  >
  > Calling Runtime::kAbort through a builtin instead of the c-entry stub
  > will allow to generate the call in a background thread, because a
  > builtin provides its own handle, whereas a code stub does not.
  >
  > @v8-mips-ports: Could you take a special look at the padding that is
  > done in MacroAssembler::Abort()?
  >
  > R=bmeurer@chromium.org, titzer@chromium.org, mstarzinger@chromium.org,
v8-mips-ports@googlegroups.com, v8-arm-ports@googlegroups.com
  >
  > Committed: https://crrev.com/9be015a254cfff871c56cd129523a729637e9158
  > Cr-Commit-Position: refs/heads/master@{#37854}

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

to

==========
X87: Revert of [builtins] Introduce a builtin for Abort(). (patchset #5 id:80001
of https://codereview.chromium.org/2156923002/ ).

  port 3e8f49ab59206b587c6c44a58eda954a76355409 (r37883)

  original commit message:
  Reason for revert:
  Blocks roll:
https://build.chromium.org/p/client.v8.ports/builders/V8%20Linux%20-%20arm64%...

  Original issue's description:
  > [builtins] Introduce a builtin for Abort().
  >
  > Calling Runtime::kAbort through a builtin instead of the c-entry stub
  > will allow to generate the call in a background thread, because a
  > builtin provides its own handle, whereas a code stub does not.
  >
  > @v8-mips-ports: Could you take a special look at the padding that is
  > done in MacroAssembler::Abort()?
  >
  > R=bmeurer@chromium.org, titzer@chromium.org, mstarzinger@chromium.org,
v8-mips-ports@googlegroups.com, v8-arm-ports@googlegroups.com
  >
  > Committed: https://crrev.com/9be015a254cfff871c56cd129523a729637e9158
  > Cr-Commit-Position: refs/heads/master@{#37854}

BUG=

Committed: https://crrev.com/870fac08c921d0f6bb42dd8b5344296a478073e8
Cr-Commit-Position: refs/heads/master@{#37888}
==========

[commit-bot: I haz the power, 2016-07-20 08:34:55]

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