Support fast-path allocation for subclass constructors with correctly initialized initial maps.

Support fast-path allocation for subclass constructors with correctly initialized initial maps.

BUG=v8:3330
LOG=n

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

[Toon Verwaest, 2015-11-09 16:06:45]

verwaest@chromium.org changed reviewers:
+ ishell@chromium.org, v8-mips-ports@googlegroups.com

[Toon Verwaest, 2015-11-09 16:06:46]

ishell: ptal
v8-mips-ports: please port to mips

[paul.l..., 2015-11-09 17:39:09]

Please find mips ports in https://codereview.chromium.org/1415393005/

[Igor Sheludko, 2015-11-10 08:08:53]

As discussed offline, for the case (constructor == original_constructor) we
don't need these new checks.

lgtm with a nit:

https://codereview.chromium.org/1413003008/diff/40001/src/arm64/builtins-arm6...
File src/arm64/builtins-arm64.cc (right):

https://codereview.chromium.org/1413003008/diff/40001/src/arm64/builtins-arm6...
src/arm64/builtins-arm64.cc:394: __ JumpIfNotObjectType(x3, x10, x11,
JS_FUNCTION_TYPE, &rt_call);
x3 -> original_constructor

[Toon Verwaest, 2015-11-10 08:13:04]

Addressed comment.

True that we don't need to check for jsfunction, but I'll defer introducing
special handling until it becomes necessary for performance.

https://codereview.chromium.org/1413003008/diff/40001/src/arm64/builtins-arm6...
File src/arm64/builtins-arm64.cc (right):

https://codereview.chromium.org/1413003008/diff/40001/src/arm64/builtins-arm6...
src/arm64/builtins-arm64.cc:394: __ JumpIfNotObjectType(x3, x10, x11,
JS_FUNCTION_TYPE, &rt_call);
On 2015/11/10 08:08:53, Igor Sheludko wrote:
> x3 -> original_constructor

Done.

[Toon Verwaest, 2015-11-10 08:13:24]

The CQ bit was checked by verwaest@chromium.org

[Toon Verwaest, 2015-11-10 08:13:24]

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

[commit-bot: I haz the power, 2015-11-10 08:13:38]

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

[commit-bot: I haz the power, 2015-11-10 08:18:52]

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

[commit-bot: I haz the power, 2015-11-10 08:18:53]

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/9967)

[Toon Verwaest, 2015-11-10 08:30:53]

The CQ bit was checked by verwaest@chromium.org

[Toon Verwaest, 2015-11-10 08:30:53]

The patchset sent to the CQ was uploaded after l-g-t-m from ishell@chromium.org
Link to the patchset: https://codereview.chromium.org/1413003008/#ps100001
(title: " ")

[commit-bot: I haz the power, 2015-11-10 08:31:07]

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

[commit-bot: I haz the power, 2015-11-10 08:50:49]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2015-11-10 08:51:29]

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