http://codereview.chromium.org/442024/diff/2001/3002
File src/ia32/codegen-ia32.cc (right):

http://codereview.chromium.org/442024/diff/2001/3002#newcode8365
src/ia32/codegen-ia32.cc:8365: void
StringAddStub::GenerateCopyAsciiCharacters(MacroAssembler* masm,
This seems inefficient at first, moving one byte at a time, but since
its only used for small counts (<13), it's probably not worth changing.
Could you add a comment saying that count is usually low?

http://codereview.chromium.org/442024

-- 
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to