[turbofan] Less aggressively insert SOFT deopts for property access.

[turbofan] Less aggressively insert SOFT deopts for property access.

Sometimes TurboFan is able to extract receiver maps from the surrounding
graph and thus is able to generate reasonable code for property accesses,
even if those haven't been executed in the baseline tier yet. So, only
stick in an SOFT deoptimization exit, if ExtractReceiverMaps failed to
infer proper receiver maps.

R=yangguo@chromium.org
BUG=v8:5267
Review-Url: https://codereview.chromium.org/2746013002
Cr-Commit-Position: refs/heads/master@{#43736}
Committed: https://chromium.googlesource.com/v8/v8/+/b8453628c94ddf71e05d70471355c7c924bbcf31

[Benedikt Meurer, 2017-03-13 06:12:59]

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

[commit-bot: I haz the power, 2017-03-13 06:13:00]

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-03-13 06:26:17]

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

[commit-bot: I haz the power, 2017-03-13 06:26:18]

Dry run: 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_...)
  v8_mac_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_mac_rel_ng/builds/18830)
  v8_mac_rel_ng_triggered on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_mac_rel_ng_triggered/bui...)

[Yang, 2017-03-13 06:30:36]

On 2017/03/13 06:12:58, Benedikt Meurer wrote:

lgtm, but tests fail :)

[Benedikt Meurer, 2017-03-13 07:03:14]

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

[commit-bot: I haz the power, 2017-03-13 07:03:19]

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

[Benedikt Meurer, 2017-03-13 07:03:51]

The CQ bit was unchecked by bmeurer@chromium.org

[Benedikt Meurer, 2017-03-13 07:03:56]

The CQ bit was checked by bmeurer@chromium.org

[Benedikt Meurer, 2017-03-13 07:03:56]

The patchset sent to the CQ was uploaded after l-g-t-m from yangguo@chromium.org
Link to the patchset: https://codereview.chromium.org/2746013002/#ps20001
(title: "Work-around for JSFunctions without context.")

[commit-bot: I haz the power, 2017-03-13 07:04:07]

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-03-13 07:23:44]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1489388636075770,
"parent_rev": "9bee8f1065da5172fa9b1746d7658a90873e47c8", "commit_rev":
"b8453628c94ddf71e05d70471355c7c924bbcf31"}

[commit-bot: I haz the power, 2017-03-13 07:23:49]

Description was changed from

==========
[turbofan] Less aggressively insert SOFT deopts for property access.

Sometimes TurboFan is able to extract receiver maps from the surrounding
graph and thus is able to generate reasonable code for property accesses,
even if those haven't been executed in the baseline tier yet. So, only
stick in an SOFT deoptimization exit, if ExtractReceiverMaps failed to
infer proper receiver maps.

R=yangguo@chromium.org
BUG=v8:5267
==========

to

==========
[turbofan] Less aggressively insert SOFT deopts for property access.

Sometimes TurboFan is able to extract receiver maps from the surrounding
graph and thus is able to generate reasonable code for property accesses,
even if those haven't been executed in the baseline tier yet. So, only
stick in an SOFT deoptimization exit, if ExtractReceiverMaps failed to
infer proper receiver maps.

R=yangguo@chromium.org
BUG=v8:5267

Review-Url: https://codereview.chromium.org/2746013002
Cr-Commit-Position: refs/heads/master@{#43736}
Committed:
https://chromium.googlesource.com/v8/v8/+/b8453628c94ddf71e05d70471355c7c924b...
==========

[commit-bot: I haz the power, 2017-03-13 07:23:51]

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

[Benedikt Meurer, 2017-03-13 11:46:43]

A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.chromium.org/2748663002/ by bmeurer@chromium.org.

The reason for reverting is: Tanks Octane/Mandreel and Octane/MandreelLatency..