Replace long running loops by OptimizeFunctionOnNextCall in some tests.

Replace long running loops by OptimizeFunctionOnNextCall in some tests that are often timing out on ARM.

[Mads Ager (chromium), 2011-05-03 13:05:08]

http://codereview.chromium.org/6910022/diff/1/test/mjsunit/closures.js
File test/mjsunit/closures.js (right):

http://codereview.chromium.org/6910022/diff/1/test/mjsunit/closures.js#newcode39
test/mjsunit/closures.js:39: %OptimizeFunctionOnNextCall(MyFunction)
Maybe call the MyFunction function a couple of times to get type feedback before
optimizing?

http://codereview.chromium.org/6910022/diff/1/test/mjsunit/compiler/regress-c...
File test/mjsunit/compiler/regress-closures-with-eval.js (right):

http://codereview.chromium.org/6910022/diff/1/test/mjsunit/compiler/regress-c...
test/mjsunit/compiler/regress-closures-with-eval.js:55:
%OptimizeFunctionOnNextCall(withEval);
Does this actually do anything? Will there be another call?

http://codereview.chromium.org/6910022/diff/1/test/mjsunit/regress/regress-10...
File test/mjsunit/regress/regress-1099.js (right):

http://codereview.chromium.org/6910022/diff/1/test/mjsunit/regress/regress-10...
test/mjsunit/regress/regress-1099.js:51: y("foo");
You should check the result here: assertTrue(y("foo"))

[Karl Klose, 2011-05-03 13:23:22]

http://codereview.chromium.org/6910022/diff/1/test/mjsunit/closures.js
File test/mjsunit/closures.js (right):

http://codereview.chromium.org/6910022/diff/1/test/mjsunit/closures.js#newcode39
test/mjsunit/closures.js:39: %OptimizeFunctionOnNextCall(MyFunction)
On 2011/05/03 13:05:08, Mads Ager wrote:
> Maybe call the MyFunction function a couple of times to get type feedback
before
> optimizing?

Done.

http://codereview.chromium.org/6910022/diff/1/test/mjsunit/compiler/regress-c...
File test/mjsunit/compiler/regress-closures-with-eval.js (right):

http://codereview.chromium.org/6910022/diff/1/test/mjsunit/compiler/regress-c...
test/mjsunit/compiler/regress-closures-with-eval.js:55:
%OptimizeFunctionOnNextCall(withEval);
Oops, call added.

http://codereview.chromium.org/6910022/diff/1/test/mjsunit/regress/regress-10...
File test/mjsunit/regress/regress-1099.js (right):

http://codereview.chromium.org/6910022/diff/1/test/mjsunit/regress/regress-10...
test/mjsunit/regress/regress-1099.js:51: y("foo");
On 2011/05/03 13:05:08, Mads Ager wrote:
> You should check the result here: assertTrue(y("foo"))

Done.

[Mads Ager (chromium), 2011-05-03 13:34:40]

LGTM