[builtins] One runtime fallback is enough for the String constructor.

[builtins] One runtime fallback is enough for the String constructor.

If inline allocation fails, we can just use the %NewObject fallback,
which will do the right thing. We don't need a dedicated fallback to
%AllocateInNewSpace.

R=verwaest@chromium.org

Committed: https://crrev.com/34b7b21d1d7bf2e9c77ae5726f868e69f3b4a4f3
Cr-Commit-Position: refs/heads/master@{#32000}

[Benedikt Meurer, 2015-11-13 12:46:43]

Hey Toon,

Here's the trivial cleanup we talked about for the StringConstructor.
Please take a look.

Thanks,
Benedikt

[Toon Verwaest, 2015-11-13 12:50:47]

lgtm

[Benedikt Meurer, 2015-11-13 12:53:24]

The CQ bit was checked by bmeurer@chromium.org

[commit-bot: I haz the power, 2015-11-13 12:53:41]

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

[commit-bot: I haz the power, 2015-11-13 12:59:29]

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

[commit-bot: I haz the power, 2015-11-13 12:59:30]

Try jobs failed on following builders:
  v8_linux_mipsel_compile_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_mipsel_compile_rel...)

[Benedikt Meurer, 2015-11-13 13:03:45]

The CQ bit was checked by bmeurer@chromium.org

[Benedikt Meurer, 2015-11-13 13:03:46]

The patchset sent to the CQ was uploaded after l-g-t-m from
verwaest@chromium.org
Link to the patchset: https://codereview.chromium.org/1440193003/#ps20001
(title: "Fix typo.")

[commit-bot: I haz the power, 2015-11-13 13:03:57]

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

[Benedikt Meurer, 2015-11-13 13:23:15]

The CQ bit was checked by bmeurer@chromium.org

[Benedikt Meurer, 2015-11-13 13:23:15]

The patchset sent to the CQ was uploaded after l-g-t-m from
verwaest@chromium.org
Link to the patchset: https://codereview.chromium.org/1440193003/#ps60001
(title: "Another fix.")

[commit-bot: I haz the power, 2015-11-13 13:23:23]

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

[commit-bot: I haz the power, 2015-11-13 13:45:22]

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

[commit-bot: I haz the power, 2015-11-13 13:45:23]

Try jobs failed on following builders:
  v8_linux_arm_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_arm_rel/builds/10104)

[Benedikt Meurer, 2015-11-13 13:53:50]

The CQ bit was checked by bmeurer@chromium.org

[Benedikt Meurer, 2015-11-13 13:53:51]

The patchset sent to the CQ was uploaded after l-g-t-m from
verwaest@chromium.org
Link to the patchset: https://codereview.chromium.org/1440193003/#ps80001
(title: "Fix scratch registers.")

[commit-bot: I haz the power, 2015-11-13 13:53:59]

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

[commit-bot: I haz the power, 2015-11-13 13:56:28]

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

[commit-bot: I haz the power, 2015-11-13 13:56:28]

Try jobs failed on following builders:
  v8_linux_mipsel_compile_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_mipsel_compile_rel...)

[Benedikt Meurer, 2015-11-15 18:59:45]

The CQ bit was checked by bmeurer@chromium.org

[Benedikt Meurer, 2015-11-15 18:59:45]

The patchset sent to the CQ was uploaded after l-g-t-m from
verwaest@chromium.org
Link to the patchset: https://codereview.chromium.org/1440193003/#ps100001
(title: "Use correct temp registers.")

[commit-bot: I haz the power, 2015-11-15 18:59:59]

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

[commit-bot: I haz the power, 2015-11-15 19:19:05]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2015-11-15 19:19:39]

Patchset 6 (id:??) landed as
https://crrev.com/34b7b21d1d7bf2e9c77ae5726f868e69f3b4a4f3
Cr-Commit-Position: refs/heads/master@{#32000}