[turbofan] Introduce map-guarded generic handler for named store.

[turbofan] Introduce map-guarded generic handler for named store.

This helps if the polymorhpic case has other maps with fast accesses.

In box2d (with high number of iterations and warm-up), we spend about 2.7% of running time in StoreIC; after this change it is only 0.2%.

Committed: https://crrev.com/225cb9fadf3c88909719645e8f93f23bfbbe2961
Cr-Commit-Position: refs/heads/master@{#40957}

[Jarin, 2016-11-10 07:37:34]

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

[commit-bot: I haz the power, 2016-11-10 07:37:44]

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

[Jarin, 2016-11-10 08:05:42]

Description was changed from

==========
[turbofan] Introduce map-guarded generic handler for named store.

This helpful if the polymorhpic case has other maps with fast accesses.
==========

to

==========
[turbofan] Introduce map-guarded generic handler for named store.

This helps if the polymorhpic case has other maps with fast accesses.

In box2d (with high number of iterations and warm-up), we spend about 2.7% of
running time in StoreIC; after this change it is only 0.2%.
==========

[Jarin, 2016-11-10 08:05:42]

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

[Jarin, 2016-11-10 08:05:55]

Could you take a look, please?

[Benedikt Meurer, 2016-11-10 08:10:05]

LGTM % nexus.

https://codereview.chromium.org/2494673002/diff/1/src/compiler/js-native-cont...
File src/compiler/js-native-context-specialization.h (right):

https://codereview.chromium.org/2494673002/diff/1/src/compiler/js-native-cont...
src/compiler/js-native-context-specialization.h:81: FeedbackNexus const& nexus,
Please pass vector+slot explicitly instead.

https://codereview.chromium.org/2494673002/diff/1/src/compiler/js-native-cont...
src/compiler/js-native-context-specialization.h:107: LanguageMode language_mode,
FeedbackNexus const& nexus);
Please pass vector+slot explicitly instead.

[commit-bot: I haz the power, 2016-11-10 08:18:37]

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

[commit-bot: I haz the power, 2016-11-10 08:18:37]

Dry run: Try jobs failed on following builders:
  v8_win_nosnap_shared_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_win_nosnap_shared_rel_ng...)
  v8_win_nosnap_shared_rel_ng_triggered on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_win_nosnap_shared_rel_ng...)

[Jarin, 2016-11-10 10:45:33]

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

[commit-bot: I haz the power, 2016-11-10 10:45:35]

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, 2016-11-10 11:11:54]

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

[commit-bot: I haz the power, 2016-11-10 11:11:55]

Dry run: This issue passed the CQ dry run.

[Benedikt Meurer, 2016-11-13 18:38:21]

LGTM, thanks.

[Jarin, 2016-11-14 09:48:14]

The CQ bit was checked by jarin@chromium.org

[Jarin, 2016-11-14 09:48:14]

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/2494673002/#ps40001
(title: "Rebase")

[commit-bot: I haz the power, 2016-11-14 09:48:24]

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, 2016-11-14 10:47:19]

Description was changed from

==========
[turbofan] Introduce map-guarded generic handler for named store.

This helps if the polymorhpic case has other maps with fast accesses.

In box2d (with high number of iterations and warm-up), we spend about 2.7% of
running time in StoreIC; after this change it is only 0.2%.
==========

to

==========
[turbofan] Introduce map-guarded generic handler for named store.

This helps if the polymorhpic case has other maps with fast accesses.

In box2d (with high number of iterations and warm-up), we spend about 2.7% of
running time in StoreIC; after this change it is only 0.2%.
==========

[commit-bot: I haz the power, 2016-11-14 10:47:19]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-11-17 22:32:12]

Description was changed from

==========
[turbofan] Introduce map-guarded generic handler for named store.

This helps if the polymorhpic case has other maps with fast accesses.

In box2d (with high number of iterations and warm-up), we spend about 2.7% of
running time in StoreIC; after this change it is only 0.2%.
==========

to

==========
[turbofan] Introduce map-guarded generic handler for named store.

This helps if the polymorhpic case has other maps with fast accesses.

In box2d (with high number of iterations and warm-up), we spend about 2.7% of
running time in StoreIC; after this change it is only 0.2%.

Committed: https://crrev.com/225cb9fadf3c88909719645e8f93f23bfbbe2961
Cr-Commit-Position: refs/heads/master@{#40957}
==========

[commit-bot: I haz the power, 2016-11-17 22:32:13]

Patchset 3 (id:??) landed as
https://crrev.com/225cb9fadf3c88909719645e8f93f23bfbbe2961
Cr-Commit-Position: refs/heads/master@{#40957}