They actually hang completely. The interpreter was borked (forgot to compile
the regexp, and was treating TheHole as a ByteArray). It's a wonder it
didn't crash.

Should be fixed now.

/L

On Mon, Mar 23, 2009 at 12:51 AM, Christian Plesner Hansen <
[email protected]> wrote:

> This change seems to have caused a fairly big performance regression
> on arm.  Before this change the longest running test took just over a
> minute, after this change 48 tests time out because they take over two
> minutes.
>
> On Sat, Mar 21, 2009 at 12:26 AM,  <[email protected]> wrote:
> >
> > Addressed issues except:
> >
> >
> > http://codereview.chromium.org/42441/diff/1008/1013
> > File test/cctest/test-api.cc (right):
> >
> > http://codereview.chromium.org/42441/diff/1008/1013#newcode5745
> > Line 5745: const char* c_source =
> > Are they more likely to fail?
> > In any case, that would be another test case, since this one doesn't do
> > GC.
> >
> > http://codereview.chromium.org/42441
> >
> > > >
> >
>



-- 
Lasse R.H. Nielsen
[email protected]
'Faith without judgement merely degrades the spirit divine'

--~--~---------~--~----~------------~-------~--~----~
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
-~----------~----~----~----~------~----~------~--~---

Reply via email to