[turbofan] Replace uninitialized JSCall nodes with SOFT deopt.

[turbofan] Replace uninitialized JSCall nodes with SOFT deopt.

Uninitialized property accesses are replaced with SOFT deopts in
TurboFan, but uninitialized JSCall nodes are not, and instead they
just stick around and are also not being inlined because the heurstic
in TurboFan doesn't consider those candidates since their call frequency
is below the threshold. This unifies the behavior and also replaces
uninitialized calls with SOFT deopts, addressing some inconsistency in
optimization behavior as discovered by Brian White of Node for example
here: https://twitter.com/mscdexdotexe/status/879005026202640385

R=jarin@chromium.org
BUG=v8:4551, v8:5267
Review-Url: https://codereview.chromium.org/2956843002
Cr-Commit-Position: refs/heads/master@{#46231}
Committed: https://chromium.googlesource.com/v8/v8/+/4ea4e7294e242583a7ec31b9a7e4f0101512e3b3

[Benedikt Meurer, 2017-06-26 09:49:04]

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

[commit-bot: I haz the power, 2017-06-26 09:49:14]

Dry run: CQ is trying da patch.

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

[Jarin, 2017-06-26 10:22:25]

lgtm

[Benedikt Meurer, 2017-06-26 10:22:42]

The CQ bit was unchecked by bmeurer@chromium.org

[Benedikt Meurer, 2017-06-26 10:22:44]

The CQ bit was checked by bmeurer@chromium.org

[commit-bot: I haz the power, 2017-06-26 10:22:53]

CQ is trying da patch.

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

[commit-bot: I haz the power, 2017-06-26 12:06:06]

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

[commit-bot: I haz the power, 2017-06-26 12:06:07]

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

[Benedikt Meurer, 2017-06-26 17:31:43]

The CQ bit was checked by bmeurer@chromium.org

[commit-bot: I haz the power, 2017-06-26 17:31:49]

CQ is trying da patch.

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

[commit-bot: I haz the power, 2017-06-26 17:44:39]

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

[commit-bot: I haz the power, 2017-06-26 17:44:40]

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

[Benedikt Meurer, 2017-06-26 19:39:20]

Description was changed from

==========
[turbofan] Replace uninitialized JSCall nodes with SOFT deopt.

Uninitialized property accesses are replaced with SOFT deopts in
TurboFan, but uninitialized JSCall nodes are not, and instead they
just stick around and are also not being inlined because the heurstic
in TurboFan doesn't consider those candidates since their call frequency
is below the threshold. This unifies the behavior and also replaces
uninitialized calls with SOFT deopts, addressing some inconsistency in
optimization behavior as discovered by Brian White of Node for example
here: https://twitter.com/mscdexdotexe/status/879005026202640385

R=jarin@chromium.org
==========

to

==========
[turbofan] Replace uninitialized JSCall nodes with SOFT deopt.

Uninitialized property accesses are replaced with SOFT deopts in
TurboFan, but uninitialized JSCall nodes are not, and instead they
just stick around and are also not being inlined because the heurstic
in TurboFan doesn't consider those candidates since their call frequency
is below the threshold. This unifies the behavior and also replaces
uninitialized calls with SOFT deopts, addressing some inconsistency in
optimization behavior as discovered by Brian White of Node for example
here: https://twitter.com/mscdexdotexe/status/879005026202640385

R=jarin@chromium.org
BUG=v8:4551, v8:5267
==========

[Benedikt Meurer, 2017-06-26 19:40:39]

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

[commit-bot: I haz the power, 2017-06-26 19:40:49]

Dry run: CQ is trying da patch.

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

[commit-bot: I haz the power, 2017-06-26 20:07:05]

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

[commit-bot: I haz the power, 2017-06-26 20:07:06]

Dry run: This issue passed the CQ dry run.

[Benedikt Meurer, 2017-06-27 03:55:38]

The CQ bit was checked by bmeurer@chromium.org

[Benedikt Meurer, 2017-06-27 03:55:39]

The patchset sent to the CQ was uploaded after l-g-t-m from jarin@chromium.org
Link to the patchset: https://codereview.chromium.org/2956843002/#ps20001
(title: "Several improvements.")

[commit-bot: I haz the power, 2017-06-27 03:55:49]

CQ is trying da patch.

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

[commit-bot: I haz the power, 2017-06-27 03:57:14]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1498535738714250,
"parent_rev": "68c6ce6707e96df480ca8be963b300bfaa63bf1c", "commit_rev":
"4ea4e7294e242583a7ec31b9a7e4f0101512e3b3"}

[commit-bot: I haz the power, 2017-06-27 03:57:22]

Description was changed from

==========
[turbofan] Replace uninitialized JSCall nodes with SOFT deopt.

Uninitialized property accesses are replaced with SOFT deopts in
TurboFan, but uninitialized JSCall nodes are not, and instead they
just stick around and are also not being inlined because the heurstic
in TurboFan doesn't consider those candidates since their call frequency
is below the threshold. This unifies the behavior and also replaces
uninitialized calls with SOFT deopts, addressing some inconsistency in
optimization behavior as discovered by Brian White of Node for example
here: https://twitter.com/mscdexdotexe/status/879005026202640385

R=jarin@chromium.org
BUG=v8:4551, v8:5267
==========

to

==========
[turbofan] Replace uninitialized JSCall nodes with SOFT deopt.

Uninitialized property accesses are replaced with SOFT deopts in
TurboFan, but uninitialized JSCall nodes are not, and instead they
just stick around and are also not being inlined because the heurstic
in TurboFan doesn't consider those candidates since their call frequency
is below the threshold. This unifies the behavior and also replaces
uninitialized calls with SOFT deopts, addressing some inconsistency in
optimization behavior as discovered by Brian White of Node for example
here: https://twitter.com/mscdexdotexe/status/879005026202640385

R=jarin@chromium.org
BUG=v8:4551, v8:5267

Review-Url: https://codereview.chromium.org/2956843002
Cr-Commit-Position: refs/heads/master@{#46231}
Committed:
https://chromium.googlesource.com/v8/v8/+/4ea4e7294e242583a7ec31b9a7e4f010151...
==========

[commit-bot: I haz the power, 2017-06-27 03:57:22]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/v8/v8/+/4ea4e7294e242583a7ec31b9a7e4f010151...