Handlify RuntimeHidden_RegExpConstructResult.

Handlify RuntimeHidden_RegExpConstructResult.

R=mvstanton@chromium.org

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

[mvstanton, 2014-04-07 14:28:49]

2 questions...if the answers are good, then 
lgtm. :)
--Michael

https://codereview.chromium.org/227593003/diff/1/src/runtime.cc
File src/runtime.cc (right):

https://codereview.chromium.org/227593003/diff/1/src/runtime.cc#newcode2607
src/runtime.cc:2607: RUNTIME_ASSERT(size >= 0 && size <=
FixedArray::kMaxLength);
So you don't bother with !Smi::IsValid(size) because it's redundant, right? (the
CONVERT_SMI_ARG_CHECKED call should do that?

https://codereview.chromium.org/227593003/diff/1/src/runtime.cc#newcode2609
src/runtime.cc:2609: RUNTIME_ASSERT(args[2]->IsSmi());
Do args[1] and args[2] actually need to be smis?

[Yang, 2014-04-07 14:47:01]

https://codereview.chromium.org/227593003/diff/1/src/runtime.cc
File src/runtime.cc (right):

https://codereview.chromium.org/227593003/diff/1/src/runtime.cc#newcode2607
src/runtime.cc:2607: RUNTIME_ASSERT(size >= 0 && size <=
FixedArray::kMaxLength);
On 2014/04/07 14:28:49, mvstanton wrote:
> So you don't bother with !Smi::IsValid(size) because it's redundant, right?
(the
> CONVERT_SMI_ARG_CHECKED call should do that?

Yes.

https://codereview.chromium.org/227593003/diff/1/src/runtime.cc#newcode2609
src/runtime.cc:2609: RUNTIME_ASSERT(args[2]->IsSmi());
On 2014/04/07 14:28:49, mvstanton wrote:
> Do args[1] and args[2] actually need to be smis?

You are right. They actually are not necessarily smis. Fixed that.

[Yang, 2014-04-08 06:38:23]

Committed patchset #2 manually as r20558 (presubmit successful).