Adding missing break from case statement. (issue6713130)

[fin...@chromium.org]

Reviewers: danno,

Description:
Adding missing break from case statement.

Doeesn't look like a correctness issue at the moment (just a mov instruction
executed twice), but could become a correctness issue in the future.

BUG=None
TEST=None
CID=15678

Please review this at http://codereview.chromium.org/6713130/

SVN Base: http://v8.googlecode.com/svn/trunk/

Affected files:
M src/ia32/stub-cache-ia32.cc

Index: src/ia32/stub-cache-ia32.cc
===================================================================
--- src/ia32/stub-cache-ia32.cc (revision 7356)
+++ src/ia32/stub-cache-ia32.cc (working copy)
@@ -3606,6 +3606,7 @@
__ bind(&done);
}
__ mov_b(Operand(edi, ebx, times_1, 0), ecx);
+ break;
case kExternalByteArray:
case kExternalUnsignedByteArray:
__ mov_b(Operand(edi, ebx, times_1, 0), ecx);

[da...@chromium.org]

LGTM

http://codereview.chromium.org/6713130/

[Finnur Thorarinsson]

Looks like I can't check this in (I'm outside corp).

Can you check this in for me?

On Mon, Mar 28, 2011 at 04:05, <da...@chromium.org> wrote:

LGTM

http://codereview.chromium.org/6713130/