Adding %_NewObject intrinsic

Adding %_NewObject intrinsic

This should help speeding up Promise and RegExp instantiations substantially.

BUG=

Committed: https://crrev.com/942f255fa2573eb3bb09390e7e213c472e03d962
Cr-Commit-Position: refs/heads/master@{#35200}

[Camillo Bruni, 2016-03-31 12:03:09]

The CQ bit was checked by cbruni@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-03-31 12:03:19]

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

[Camillo Bruni, 2016-03-31 12:04:23]

cbruni@chromium.org changed reviewers:
+ bmeurer@chromium.org

[Camillo Bruni, 2016-03-31 12:04:24]

PTAL some low-hanging fruit here ;)

[commit-bot: I haz the power, 2016-03-31 12:25:50]

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

[commit-bot: I haz the power, 2016-03-31 12:25:50]

Dry run: This issue passed the CQ dry run.

[Benedikt Meurer, 2016-03-31 17:49:29]

Nice. Looking good, just two lowering changes.

https://codereview.chromium.org/1850643002/diff/1/src/compiler/js-intrinsic-l...
File src/compiler/js-intrinsic-lowering.cc (right):

https://codereview.chromium.org/1850643002/diff/1/src/compiler/js-intrinsic-l...
src/compiler/js-intrinsic-lowering.cc:483: return Change(node,
CodeFactory::FastNewObject(isolate()), 0);
%_NewObject should be lowered to JSCreate, which does the fast case if possible
and otherwise falls back to FastNewObjectStub.

https://codereview.chromium.org/1850643002/diff/1/src/full-codegen/full-codeg...
File src/full-codegen/full-codegen.cc (right):

https://codereview.chromium.org/1850643002/diff/1/src/full-codegen/full-codeg...
src/full-codegen/full-codegen.cc:511: // Load the arguments on the stack and
call the stub.
Please use EmitIntrinsicAsStubCall here.

[Camillo Bruni, 2016-04-01 10:44:13]

The CQ bit was checked by cbruni@chromium.org

[Camillo Bruni, 2016-04-01 10:44:14]

PTAL again

https://codereview.chromium.org/1850643002/diff/1/src/full-codegen/full-codeg...
File src/full-codegen/full-codegen.cc (right):

https://codereview.chromium.org/1850643002/diff/1/src/full-codegen/full-codeg...
src/full-codegen/full-codegen.cc:511: // Load the arguments on the stack and
call the stub.
On 2016/03/31 at 17:49:29, Benedikt Meurer wrote:
> Please use EmitIntrinsicAsStubCall here.

Could swear that I tried that :D

https://codereview.chromium.org/1850643002/diff/20001/src/compiler/js-intrins...
File src/compiler/js-intrinsic-lowering.cc (right):

https://codereview.chromium.org/1850643002/diff/20001/src/compiler/js-intrins...
src/compiler/js-intrinsic-lowering.cc:484: return Changed(node);
Check failed: OperatorProperties::GetTotalInputCount(node->op()) ==
node->InputCount() (5 vs. 6).

[commit-bot: I haz the power, 2016-04-01 10:44:22]

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

[commit-bot: I haz the power, 2016-04-01 10:44:23]

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

[commit-bot: I haz the power, 2016-04-01 10:44:24]

No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
Committers are members of the group "project-v8-committers".
Note that this has nothing to do with OWNERS files.

[Benedikt Meurer, 2016-04-01 10:56:44]

LGTM, thanks.

[Michael Starzinger, 2016-04-01 10:57:02]

https://codereview.chromium.org/1850643002/diff/60001/src/compiler/linkage.cc
File src/compiler/linkage.cc (right):

https://codereview.chromium.org/1850643002/diff/60001/src/compiler/linkage.cc...
src/compiler/linkage.cc:174: case Runtime::kInlineNewObject:
nit: Please keep list alpha-sorted.

[Camillo Bruni, 2016-04-01 11:06:37]

The CQ bit was checked by cbruni@chromium.org

[Camillo Bruni, 2016-04-01 11:06:37]

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

[commit-bot: I haz the power, 2016-04-01 11:06:48]

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

[commit-bot: I haz the power, 2016-04-01 11:33:59]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-04-01 11:34:31]

Description was changed from

==========
Adding %_NewObject intrinsic

This should help speeding up Promise and RegExp instantiations substantially.

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

to

==========
Adding %_NewObject intrinsic

This should help speeding up Promise and RegExp instantiations substantially.

BUG=

Committed: https://crrev.com/942f255fa2573eb3bb09390e7e213c472e03d962
Cr-Commit-Position: refs/heads/master@{#35200}
==========

[commit-bot: I haz the power, 2016-04-01 11:34:32]

Patchset 5 (id:??) landed as
https://crrev.com/942f255fa2573eb3bb09390e7e213c472e03d962
Cr-Commit-Position: refs/heads/master@{#35200}