Handlify CompileConstructStub.

Handlify CompileConstructStub. Based on 8391045.

R=kmillikin@chromium.org
BUG=
TEST=

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

[ulan, 2011-10-27 11:37:37]

Please take a look.

http://codereview.chromium.org/8399032/diff/1/src/runtime.cc
File src/runtime.cc (right):

http://codereview.chromium.org/8399032/diff/1/src/runtime.cc#newcode8181
src/runtime.cc:8181: static void TrySettingInlineConstructStub(Isolate* isolate,
The "Try" in the name bothers me, but I cannot remove it because the stub is set
or not set depending on CanGenerateInlineConstructor.

[Kevin Millikin (Chromium), 2011-10-27 12:35:24]

LGTM with one small change noted.

http://codereview.chromium.org/8399032/diff/1/src/runtime.cc
File src/runtime.cc (right):

http://codereview.chromium.org/8399032/diff/1/src/runtime.cc#newcode8181
src/runtime.cc:8181: static void TrySettingInlineConstructStub(Isolate* isolate,
On 2011/10/27 11:37:37, ulan wrote:
> The "Try" in the name bothers me, but I cannot remove it because the stub is
set
> or not set depending on CanGenerateInlineConstructor.

That's OK, it's a different kind of "Try".  There is no allocation failure.

http://codereview.chromium.org/8399032/diff/1/src/runtime.cc#newcode8191
src/runtime.cc:8191: Code::cast(code->ToObjectUnchecked()));
...->set_construct_stub(*code);