Enable all vm tests on simarm (skip 2 timing out byte array tests).

Enable all vm tests on simarm (skip 2 timing out byte array tests).
Reduce number of iterations in inline_stack_frame_test to avoid time out.

R=zra@google.com

Committed: https://code.google.com/p/dart/source/detail?r=22280

[siva, 2013-04-30 17:24:21]

DBC

https://codereview.chromium.org/14651005/diff/1/runtime/tests/vm/dart/inline_...
File runtime/tests/vm/dart/inline_stack_frame_test.dart (right):

https://codereview.chromium.org/14651005/diff/1/runtime/tests/vm/dart/inline_...
runtime/tests/vm/dart/inline_stack_frame_test.dart:69: }
Maybe add

// VMOptions=--optimization_counter_threshold=10
to the top of the test.

You could change this loop iteration to even lower maybe 10.

https://codereview.chromium.org/14651005/diff/1/runtime/tests/vm/vm.status
File runtime/tests/vm/vm.status (right):

https://codereview.chromium.org/14651005/diff/1/runtime/tests/vm/vm.status#ne...
runtime/tests/vm/vm.status:65: dart/byte_array_optimized_test: Skip
maybe you could use the same strategy of specifying the threshold to 10 and
lowering the loop iteration count to 10 instead of skipping the test.

[zra, 2013-04-30 18:01:39]

lgtm with Siva's suggestions.

https://codereview.chromium.org/14651005/diff/1/runtime/tests/vm/dart/inline_...
File runtime/tests/vm/dart/inline_stack_frame_test.dart (right):

https://codereview.chromium.org/14651005/diff/1/runtime/tests/vm/dart/inline_...
runtime/tests/vm/dart/inline_stack_frame_test.dart:69: }
On 2013/04/30 17:24:21, siva wrote:
> Maybe add
> 
> // VMOptions=--optimization_counter_threshold=10
> to the top of the test.
> 
> You could change this loop iteration to even lower maybe 10.

If this loop counter is made this much smaller, then the bound in func5 (1030)
probably also needs to change.

[regis, 2013-05-01 21:09:46]

Thanks both!

After adding the optimization counter flag to inline_stack_frame_test, the code
started to optimize/deoptimize, which is not supported yet.

So I am leaving this edited test as skipped for now and I will address the
different issues I encountered in following cls.

-- Regis

https://codereview.chromium.org/14651005/diff/1/runtime/tests/vm/dart/inline_...
File runtime/tests/vm/dart/inline_stack_frame_test.dart (right):

https://codereview.chromium.org/14651005/diff/1/runtime/tests/vm/dart/inline_...
runtime/tests/vm/dart/inline_stack_frame_test.dart:69: }
On 2013/04/30 17:24:21, siva wrote:
> Maybe add
> 
> // VMOptions=--optimization_counter_threshold=10
> to the top of the test.
> 
> You could change this loop iteration to even lower maybe 10.

Done.

https://codereview.chromium.org/14651005/diff/1/runtime/tests/vm/dart/inline_...
runtime/tests/vm/dart/inline_stack_frame_test.dart:69: }
On 2013/04/30 18:01:39, zra wrote:
> On 2013/04/30 17:24:21, siva wrote:
> > Maybe add
> > 
> > // VMOptions=--optimization_counter_threshold=10
> > to the top of the test.
> > 
> > You could change this loop iteration to even lower maybe 10.
> 
> If this loop counter is made this much smaller, then the bound in func5 (1030)
> probably also needs to change.

Yes, I had to reduce it to 520.

https://codereview.chromium.org/14651005/diff/1/runtime/tests/vm/vm.status
File runtime/tests/vm/vm.status (right):

https://codereview.chromium.org/14651005/diff/1/runtime/tests/vm/vm.status#ne...
runtime/tests/vm/vm.status:65: dart/byte_array_optimized_test: Skip
On 2013/04/30 17:24:21, siva wrote:
> maybe you could use the same strategy of specifying the threshold to 10 and
> lowering the loop iteration count to 10 instead of skipping the test.

Looking at the tests, we may have other issues. Indexed access is not fully
implemented yet. I'll investigate this in a following cl. Leaving the tests as
skipped for now, but keeping your good suggestion in mind.

[regis, 2013-05-01 21:10:03]

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