Vector-based ICs also need to hold maps weakly.

Vector-based ICs also need to hold maps weakly.

Regular ICs in MONOMORPHIC and POLYMORPHIC state now hold onto maps with
WeakCells. Vector-based ICs should do the same.

R=ulan@chromium.org

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

[mvstanton, 2014-12-19 12:51:31]

Hi Ulan,
This allows vector-ics to be weakly walked. Note that CALL_IC doesn't change
it's behavior, but that is okay because it is always cleared on GCs. The
vector-based LOAD_IC and KEYED_LOAD_IC are now in conformance with their
patch-based brethren.

This CL can land after your fix and also
https://codereview.chromium.org/820673002/ which has been submitted.

Thanks,
--Michael

[mvstanton, 2014-12-19 14:15:04]

Hi Ulan,
Here is a rebase and fix for the comments. I didn't see your change in
hydrogen-instructions.h after all?
--Michael

[ulan, 2014-12-19 14:30:16]

LGTM

> I didn't see your change in hydrogen-instructions.h after all?
Ah, right, just saw that it is not landed yet. Added to CQ (812353002).

[mvstanton, 2014-12-19 16:26:18]

The CQ bit was checked by mvstanton@chromium.org

[commit-bot: I haz the power, 2014-12-19 16:27:06]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/815953002/40001

[commit-bot: I haz the power, 2014-12-19 16:53:47]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2014-12-19 16:53:57]

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