store ics for js api accessors

store ics for js api accessors

R=mstarzinger@chromium.org
BUG=

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

[dcarney, 2013-09-05 13:17:33]

ptal

[Michael Starzinger, 2013-09-05 16:24:49]

LGTM. Can we haz test coverage?

https://codereview.chromium.org/23549019/diff/1/src/ia32/stub-cache-ia32.cc
File src/ia32/stub-cache-ia32.cc (left):

https://codereview.chromium.org/23549019/diff/1/src/ia32/stub-cache-ia32.cc#o...
src/ia32/stub-cache-ia32.cc:1359:
ASSERT(call_optimization.is_simple_api_call());
nit: Can we preserve this assert in this function? Probably applies to all
architectures.

[dcarney, 2013-09-05 19:25:46]

yep, will add tests

https://codereview.chromium.org/23549019/diff/1/src/ia32/stub-cache-ia32.cc
File src/ia32/stub-cache-ia32.cc (left):

https://codereview.chromium.org/23549019/diff/1/src/ia32/stub-cache-ia32.cc#o...
src/ia32/stub-cache-ia32.cc:1359:
ASSERT(call_optimization.is_simple_api_call());
On 2013/09/05 16:24:49, Michael Starzinger wrote:
> nit: Can we preserve this assert in this function? Probably applies to all
> architectures.

yep, I thought I moved it into GenerateFastApiCall everywhere, but apparently it
got lost in the shuffle

[dcarney, 2013-09-06 11:32:33]

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