diff options
| author | Nicolas Geoffray <ngeoffray@google.com> | 2015-01-16 09:04:49 +0000 |
|---|---|---|
| committer | Nicolas Geoffray <ngeoffray@google.com> | 2015-01-16 09:08:07 +0000 |
| commit | 6450d14f4f3da6fcb519a0bb948acc8cd274d5a3 (patch) | |
| tree | 3b56f57b40ae035fc73629a347d15d083a0a0b20 /compiler/optimizing/parallel_move_test.cc | |
| parent | 6a5f5b25b3866966175859bc20216c5519d8b029 (diff) | |
Remove test that is now invalid.
It does not make sense to have moves to the same destination
within a single parallel move. There is now an explicit
DCHECK to prevent these situations.
Change-Id: I3a33e748579ecb53b95476e77a37164d2f145f5a
Diffstat (limited to 'compiler/optimizing/parallel_move_test.cc')
| -rw-r--r-- | compiler/optimizing/parallel_move_test.cc | 11 |
1 files changed, 2 insertions, 9 deletions
diff --git a/compiler/optimizing/parallel_move_test.cc b/compiler/optimizing/parallel_move_test.cc index 210f7d7f09..7ab41b6ddc 100644 --- a/compiler/optimizing/parallel_move_test.cc +++ b/compiler/optimizing/parallel_move_test.cc @@ -120,16 +120,9 @@ TEST(ParallelMoveTest, Swap) { { TestParallelMoveResolver resolver(&allocator); - static constexpr size_t moves[][2] = {{0, 1}, {1, 2}, {2, 3}, {3, 4}, {4, 1}}; + static constexpr size_t moves[][2] = {{0, 1}, {1, 2}, {2, 3}, {3, 4}, {4, 0}}; resolver.EmitNativeCode(BuildParallelMove(&allocator, moves, arraysize(moves))); - ASSERT_STREQ("(4 <-> 1) (3 <-> 4) (2 <-> 3) (0 -> 1)", resolver.GetMessage().c_str()); - } - - { - TestParallelMoveResolver resolver(&allocator); - static constexpr size_t moves[][2] = {{0, 1}, {1, 2}, {2, 3}, {3, 4}, {4, 1}, {5, 4}}; - resolver.EmitNativeCode(BuildParallelMove(&allocator, moves, arraysize(moves))); - ASSERT_STREQ("(4 <-> 1) (3 <-> 4) (2 <-> 3) (0 -> 1) (5 -> 4)", resolver.GetMessage().c_str()); + ASSERT_STREQ("(4 <-> 0) (3 <-> 4) (2 <-> 3) (1 <-> 2)", resolver.GetMessage().c_str()); } } |