Comment the code for loading and calling with interceptor.

Comment the code for loading and calling with interceptor.

Committed: http://code.google.com/p/v8/source/detail?r=4717

[antonm, 2010-05-25 13:42:56]

Søren,

may you have a look?

[Søren Thygesen Gjesse, 2010-05-25 14:07:28]

LGTM

http://codereview.chromium.org/2124021/diff/2001/3001
File src/arm/stub-cache-arm.cc (right):

http://codereview.chromium.org/2124021/diff/2001/3001#newcode787
src/arm/stub-cache-arm.cc:787: // holder haven't changed and thus we can use
cached constant function.
I am wondering whether this check can be skipped if the interceptor holder and
the lookup holder are the same, or is that not possible?

[antonm, 2010-05-25 14:15:09]

Thanks for review, committing.

http://codereview.chromium.org/2124021/diff/2001/3001
File src/arm/stub-cache-arm.cc (right):

http://codereview.chromium.org/2124021/diff/2001/3001#newcode787
src/arm/stub-cache-arm.cc:787: // holder haven't changed and thus we can use
cached constant function.
On 2010/05/25 14:07:29, Søren Gjesse wrote:
> I am wondering whether this check can be skipped if the interceptor holder and
> the lookup holder are the same, or is that not possible?

That's a good point and it indeed could happen.  Let me address that with a
separate CL.