On 2013/09/17 10:35:44, Benedikt Meurer wrote:
On 2013/09/17 09:21:43, Bangfu wrote:
> On 2013/09/17 09:01:20, ulan wrote:
> > Committed patchset #4 manually as r16751 (presubmit successful).
>
> Thanks a lot.
The test does not pass in arm.debug, we had to revert.
There is a bug in the current copyBytes(), which is fixed in my previous
patch.
Now my test case tricked the bug.
Please look at the code, the assert occurred with my new test case:
// Copy bytes in word size chunks.
if (emit_debug_code()) {
tst(src, Operand(kPointerSize - 1));
Assert(eq, kExpectingAlignmentForCopyBytes);
}
The assert would occur if length is 1 and src address ends with "01 or 10".
Do you want me to fix the bug in copyBytes or use my previous patch instead?
https://codereview.chromium.org/23480027/
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.
