Load target types and handlers before IC computation.

Load target types and handlers before IC computation.

BUG=
R=verwaest@chromium.org

Committed: https://code.google.com/p/v8/source/detail?r=19455

[ulan, 2014-02-18 14:50:26]

Toon, ptal, this is the refactoring we discussed that prepares IC to handle weak
maps.

https://codereview.chromium.org/170563002/diff/1/src/ic.cc
File src/ic.cc (right):

https://codereview.chromium.org/170563002/diff/1/src/ic.cc#newcode635
src/ic.cc:635: handlers()->Set(handler_to_overwrite, code);
This function modifies handlers and types. Do you think copying the lists here
would be better?

[Toon Verwaest, 2014-02-18 15:06:36]

lgtm

https://codereview.chromium.org/170563002/diff/1/src/ic.cc
File src/ic.cc (right):

https://codereview.chromium.org/170563002/diff/1/src/ic.cc#newcode635
src/ic.cc:635: handlers()->Set(handler_to_overwrite, code);
This is done at the end of IC computation. It shouldn't matter anymore
afterwards. We shouldn't care anymore about handlers that were there beforehand.

On 2014/02/18 14:50:26, ulan wrote:
> This function modifies handlers and types. Do you think copying the lists here
> would be better?

[ulan, 2014-02-18 15:13:04]

Committed patchset #1 manually as r19455 (presubmit successful).