[stubs] Cleanup storing of maps to objects.

[stubs] Cleanup storing of maps to objects.

1) CSA::StoreMap() added which triggers map-specific write barrier
   unlike StoreObjectField() which triggers the full write barrier.
2) CSA::StoreMapNoWriteBarrier(object, map_root_index) added.
3) StoreMapNoWriteBarrier() is used for storing immortal immovable
   maps since they don't require write barriers even for objects in
   old space.

BUG=

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

[Igor Sheludko, 2016-12-01 12:51:06]

Patchset #1 (id:1) has been deleted

[Igor Sheludko, 2016-12-01 12:51:51]

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

[commit-bot: I haz the power, 2016-12-01 12:52:01]

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

[Igor Sheludko, 2016-12-01 12:52:43]

ishell@chromium.org changed reviewers:
+ ulan@chromium.org

[Igor Sheludko, 2016-12-01 12:52:44]

PTAL

[commit-bot: I haz the power, 2016-12-01 13:18:14]

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

[commit-bot: I haz the power, 2016-12-01 13:18:15]

Dry run: This issue passed the CQ dry run.

[Igor Sheludko, 2016-12-01 15:40:47]

Patchset #1 (id:20001) has been deleted

[Igor Sheludko, 2016-12-01 15:54:11]

Patchset #2 (id:60001) has been deleted

[Igor Sheludko, 2016-12-01 15:54:17]

Patchset #1 (id:40001) has been deleted

[Igor Sheludko, 2016-12-01 15:55:49]

Description was changed from

==========
[stubs] Cleanup storing of maps to objects.

This CL removes CSA::StoreMapNoWriteBarrier() and instead introduces
1) CSA::StoreMapRoot() for storing root maps and
2) CSA::StoreMap() for storing arbitrary maps.

The former one may skil the write barrier if the map is immortal immovable
while the latter always triggers map-specific write barrier.

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

to

==========
[stubs] Cleanup storing of maps to objects.

This CL adds CSA::StoreMap() which triggers map-specific write barrier unlike
StoreObjectField() which triggers the full write barrier.

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

[Igor Sheludko, 2016-12-01 15:57:38]

Description was changed from

==========
[stubs] Cleanup storing of maps to objects.

This CL adds CSA::StoreMap() which triggers map-specific write barrier unlike
StoreObjectField() which triggers the full write barrier.

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

to

==========
[stubs] Cleanup storing of maps to objects.

1) CSA::StoreMap() added which triggers map-specific write barrier unlike
StoreObjectField() which triggers the full write barrier.
2) CSA::StoreMapNoWriteBarrier(object, map_root_index) added.

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

[Igor Sheludko, 2016-12-01 16:02:45]

Description was changed from

==========
[stubs] Cleanup storing of maps to objects.

1) CSA::StoreMap() added which triggers map-specific write barrier unlike
StoreObjectField() which triggers the full write barrier.
2) CSA::StoreMapNoWriteBarrier(object, map_root_index) added.

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

to

==========
[stubs] Cleanup storing of maps to objects.

1) CSA::StoreMap() added which triggers map-specific write barrier
   unlike StoreObjectField() which triggers the full write barrier.
2) CSA::StoreMapNoWriteBarrier(object, map_root_index) added.
3) StoreMapNoWriteBarrier() is used for storing immortal immovable
   maps since they don't require write barriers even for objects in
   old space.

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

[Igor Sheludko, 2016-12-01 16:07:18]

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

[commit-bot: I haz the power, 2016-12-01 16:07:27]

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-12-01 16:32:22]

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

[commit-bot: I haz the power, 2016-12-01 16:32:22]

Dry run: This issue passed the CQ dry run.

[Michael Starzinger, 2016-12-01 16:40:56]

mstarzinger@chromium.org changed reviewers:
+ mstarzinger@chromium.org

[Michael Starzinger, 2016-12-01 16:40:57]

LGTM on "interpreter", didn't look at the rest.

[ulan, 2016-12-01 16:48:50]

lgtm!

[Igor Sheludko, 2016-12-01 16:49:18]

The CQ bit was checked by ishell@chromium.org

[commit-bot: I haz the power, 2016-12-01 16:49:29]

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-12-01 16:51:09]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1480610958436800,
"parent_rev": "eafca29c9b01378fa7cb9e613c3c8707e1af7f8b", "commit_rev":
"3243e0e0f9d13b53499ed1ace7e69115328c6173"}

[commit-bot: I haz the power, 2016-12-01 16:51:18]

Description was changed from

==========
[stubs] Cleanup storing of maps to objects.

1) CSA::StoreMap() added which triggers map-specific write barrier
   unlike StoreObjectField() which triggers the full write barrier.
2) CSA::StoreMapNoWriteBarrier(object, map_root_index) added.
3) StoreMapNoWriteBarrier() is used for storing immortal immovable
   maps since they don't require write barriers even for objects in
   old space.

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

to

==========
[stubs] Cleanup storing of maps to objects.

1) CSA::StoreMap() added which triggers map-specific write barrier
   unlike StoreObjectField() which triggers the full write barrier.
2) CSA::StoreMapNoWriteBarrier(object, map_root_index) added.
3) StoreMapNoWriteBarrier() is used for storing immortal immovable
   maps since they don't require write barriers even for objects in
   old space.

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

[commit-bot: I haz the power, 2016-12-01 16:51:20]

Committed patchset #1 (id:80001)

[commit-bot: I haz the power, 2016-12-01 16:51:48]

Description was changed from

==========
[stubs] Cleanup storing of maps to objects.

1) CSA::StoreMap() added which triggers map-specific write barrier
   unlike StoreObjectField() which triggers the full write barrier.
2) CSA::StoreMapNoWriteBarrier(object, map_root_index) added.
3) StoreMapNoWriteBarrier() is used for storing immortal immovable
   maps since they don't require write barriers even for objects in
   old space.

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

to

==========
[stubs] Cleanup storing of maps to objects.

1) CSA::StoreMap() added which triggers map-specific write barrier
   unlike StoreObjectField() which triggers the full write barrier.
2) CSA::StoreMapNoWriteBarrier(object, map_root_index) added.
3) StoreMapNoWriteBarrier() is used for storing immortal immovable
   maps since they don't require write barriers even for objects in
   old space.

BUG=

Committed: https://crrev.com/d4c3fa8d2418c81e0ba06fb7d8b24cff25a7a55e
Cr-Commit-Position: refs/heads/master@{#41435}
==========

[commit-bot: I haz the power, 2016-12-01 16:51:49]

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