Ensure that ToPositiveInteger is optimizable.

Ensure that ToPositiveInteger is optimizable.

'throw %NAME(..)' cause hydrogen optimizations to be disabled for
surrounding function. This patch rectifies.

R=yangguo@chromium.org

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

[Dmitry Lomov (no reviews), 2013-11-13 15:34:27]

PTAL. Gives some speedup for typed arrays and friends

[Dmitry Lomov (no reviews), 2013-11-15 18:20:02]

On 2013/11/13 15:34:27, Dmitry Lomov (chromium) wrote:
> PTAL. Gives some speedup for typed arrays and friends

ping

[Yang, 2013-11-18 08:53:03]

On 2013/11/15 18:20:02, Dmitry Lomov (chromium) wrote:
> On 2013/11/13 15:34:27, Dmitry Lomov (chromium) wrote:
> > PTAL. Gives some speedup for typed arrays and friends
> 
> ping

I wonder why we have this at all. We also have a bunch of %MakeTypeError here
and there. Are those necessary or can we also change them in the same way?

[Yang, 2013-11-18 08:53:41]

On 2013/11/18 08:53:03, Yang wrote:
> On 2013/11/15 18:20:02, Dmitry Lomov (chromium) wrote:
> > On 2013/11/13 15:34:27, Dmitry Lomov (chromium) wrote:
> > > PTAL. Gives some speedup for typed arrays and friends
> > 
> > ping
> 
> I wonder why we have this at all. We also have a bunch of %MakeTypeError here
> and there. Are those necessary or can we also change them in the same way?

LGTM btw.

[Dmitry Lomov (no reviews), 2013-11-18 08:56:57]

On 2013/11/18 08:53:03, Yang wrote:
> On 2013/11/15 18:20:02, Dmitry Lomov (chromium) wrote:
> > On 2013/11/13 15:34:27, Dmitry Lomov (chromium) wrote:
> > > PTAL. Gives some speedup for typed arrays and friends
> > 
> > ping
> 
> I wonder why we have this at all. We also have a bunch of %MakeTypeError here
> and there. Are those necessary or can we also change them in the same way?

I actually do not know. Looks like we can lose a % here.

[Dmitry Lomov (no reviews), 2013-11-18 14:07:31]

Committed patchset #1 manually as r17832 (presubmit successful).