Start to add assembler tests for DBC

Start to add assembler tests for DBC

R=iposva@google.com

Committed: https://github.com/dart-lang/sdk/commit/7c9fe494950508ae268f3eb1a8ed40c336b91ee6

[zra, 2016-04-28 19:24:38]

zra@google.com changed reviewers:
+ iposva@google.com, vegorov@google.com

[Ivan Posva, 2016-04-29 05:57:53]

LGTM -ip

https://codereview.chromium.org/1927203002/diff/100001/runtime/vm/assembler_d...
File runtime/vm/assembler_dbc_test.cc (right):

https://codereview.chromium.org/1927203002/diff/100001/runtime/vm/assembler_d...
runtime/vm/assembler_dbc_test.cc:37: // TODO(zra): Test fail over to the runtime
call.
You have this TODO here, but not in the other places such as SubTOS.

[zra, 2016-04-29 16:08:57]

https://codereview.chromium.org/1927203002/diff/100001/runtime/vm/assembler_d...
File runtime/vm/assembler_dbc_test.cc (right):

https://codereview.chromium.org/1927203002/diff/100001/runtime/vm/assembler_d...
runtime/vm/assembler_dbc_test.cc:37: // TODO(zra): Test fail over to the runtime
call.
On 2016/04/29 05:57:53, Ivan Posva wrote:
> You have this TODO here, but not in the other places such as SubTOS.

On further thought, I think this is not the right place to test this, so I've
removed the TODO. All that should be tested here is that the next instruction is
skipped when the operands are Smis and there is no overflow.

[zra, 2016-04-29 16:09:45]

Description was changed from

==========
Start to add assembler tests for DBC
==========

to

==========
Start to add assembler tests for DBC

R=iposva@google.com

Committed:
https://github.com/dart-lang/sdk/commit/7c9fe494950508ae268f3eb1a8ed40c336b91ee6
==========

[zra, 2016-04-29 16:09:46]

Committed patchset #7 (id:120001) manually as
7c9fe494950508ae268f3eb1a8ed40c336b91ee6 (presubmit successful).