Fixed a bug in the chaining of fixup position
authorsvenpanne@chromium.org <svenpanne@chromium.org@ce2b1a6d-e550-0410-aec6-3dcde31c8c00>
Tue, 30 Aug 2011 07:36:31 +0000 (07:36 +0000)
committersvenpanne@chromium.org <svenpanne@chromium.org@ce2b1a6d-e550-0410-aec6-3dcde31c8c00>
Tue, 30 Aug 2011 07:36:31 +0000 (07:36 +0000)
commit4084e698c3e1240088c8b9370909008479d12854
tree818f3ce67a64d9b0f1a12b10c0fe5b9cbca62cac
parentd74cceb2bc5b773ae3c68d44380318bbf4599ac2
Fixed a bug in the chaining of fixup position

The ARM and MIPS assemblers had a bug where they did not handle the last element
in the list of code positions correctly during the fixup of offsets for forward
jumps. This happened when the first instruction contained a forward jump to a
label, and that label was used in a forward jump later, too.

Unified the code for Assembler::next on ARM and MIPS while we were there.

Added test cases, even for ia32/x64, which seem to be correct, even I don't
fully understand why... %-}

BUG=v8:1644
Review URL: http://codereview.chromium.org/7786001

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9063 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
src/arm/assembler-arm.cc
src/mips/assembler-mips.cc
test/cctest/test-assembler-arm.cc
test/cctest/test-assembler-ia32.cc
test/cctest/test-assembler-mips.cc
test/cctest/test-assembler-x64.cc