VectorICs: refactoring to eliminate "for queries only" vector ic mode.

VectorICs: refactoring to eliminate "for queries only" vector ic mode.

Since we need the notion of a dummy vector ic, we can use that to avoid
a special case of the IC constructor. Also, consolidate the two dummy
ICs into one.

BUG=

Committed: https://crrev.com/1a5751f9b3ca682fadb6fce8202dda2db5b017c6
Cr-Commit-Position: refs/heads/master@{#29956}

[mvstanton, 2015-07-31 12:33:17]

Patchset #1 (id:1) has been deleted

[mvstanton, 2015-07-31 12:38:04]

mvstanton@chromium.org changed reviewers:
+ yangguo@chromium.org

[mvstanton, 2015-07-31 12:38:04]

Thanks for the look, Yang!

[Yang, 2015-07-31 13:14:58]

looking good so far. Maybe you can define the dummy as TypeFeedbackVector
directly so you don't need the cast helper.

[mvstanton, 2015-07-31 13:26:19]

Ugh. It breaks the world because the ROOT_LIST needs the complete definition of
TypeFeedbackVector and a circular dependency results. I'm playing with it, but
doesn't look awesome...

[Yang, 2015-07-31 13:44:18]

On 2015/07/31 13:26:19, mvstanton wrote:
> Ugh. It breaks the world because the ROOT_LIST needs the complete definition
of
> TypeFeedbackVector and a circular dependency results. I'm playing with it, but
> doesn't look awesome...

lgtm.

[mvstanton, 2015-07-31 13:45:22]

The CQ bit was checked by mvstanton@chromium.org

[commit-bot: I haz the power, 2015-07-31 13:45:28]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1268783004/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1268783004/60001

[commit-bot: I haz the power, 2015-07-31 14:04:06]

Committed patchset #3 (id:60001)

[commit-bot: I haz the power, 2015-07-31 14:04:20]

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