Do not cache CodeStubInterfaceDescriptor on the isolate.

Do not cache CodeStubInterfaceDescriptor on the isolate.

R=mvstanton@chromium.org

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

[mvstanton, 2014-09-05 13:38:44]

lgtm

https://codereview.chromium.org/544123002/diff/1/src/code-stubs.cc
File src/code-stubs.cc (right):

https://codereview.chromium.org/544123002/diff/1/src/code-stubs.cc#newcode201
src/code-stubs.cc:201: void CodeStub::Dispatch(Isolate* isolate, uint32_t key,
void** value_out,
As we discussed, this is a better solution than making code stubs heap
allocatable. It's good that you keep it internal to the code stub class.

https://codereview.chromium.org/544123002/diff/1/src/code-stubs.h
File src/code-stubs.h (left):

https://codereview.chromium.org/544123002/diff/1/src/code-stubs.h#oldcode136
src/code-stubs.h:136: UninitializedMajorKey = 0,
yay, this is gone! :)

https://codereview.chromium.org/544123002/diff/1/src/ia32/lithium-ia32.cc
File src/ia32/lithium-ia32.cc (right):

https://codereview.chromium.org/544123002/diff/1/src/ia32/lithium-ia32.cc#new...
src/ia32/lithium-ia32.cc:2504: CodeStubInterfaceDescriptor descriptor;
It will be great when you allow writing:

CodeStubInterfaceDescriptor descriptor(info()->code_stub());

Looking forward to that soon!

[Yang, 2014-09-05 15:21:05]

Committed patchset #1 (id:1) manually as 23744 (presubmit successful).