StubCompiler gets extra_ic_state member.

Removed pieces of extra_ic_state scattered around the stub compiler class hierarchy, and pushed down to a extra_ic_state in the base class.

R=verwaest@chromium.org

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

[mvstanton, 2013-12-02 11:08:45]

PTAL, thx!
--Michael

[Toon Verwaest, 2013-12-02 11:44:31]

I like this change. Just one comment.

https://codereview.chromium.org/98853002/diff/20001/src/stub-cache.h
File src/stub-cache.h (right):

https://codereview.chromium.org/98853002/diff/20001/src/stub-cache.h#newcode840
src/stub-cache.h:840: return StoreIC::GetStrictMode(extra_state());
Is this method used?

[mvstanton, 2013-12-02 11:48:50]

Hi Toon,
Thx for comments. Indeed, I could remove strict_mode().
--Michael

https://codereview.chromium.org/98853002/diff/20001/src/stub-cache.h
File src/stub-cache.h (right):

https://codereview.chromium.org/98853002/diff/20001/src/stub-cache.h#newcode840
src/stub-cache.h:840: return StoreIC::GetStrictMode(extra_state());
On 2013/12/02 11:44:31, Toon Verwaest wrote:
> Is this method used?

I removed that one now, good catch.

[Toon Verwaest, 2013-12-02 11:50:16]

lgtm.

[mvstanton, 2013-12-02 11:59:54]

Committed patchset #3 manually as r18185 (presubmit successful).