Quick comment:

http://codereview.chromium.org/2101002/diff/2001/3015
File src/ia32/macro-assembler-ia32.cc (right):

http://codereview.chromium.org/2101002/diff/2001/3015#newcode67
src/ia32/macro-assembler-ia32.cc:67: bts(Operand(scratch), addr);
Can't you use:

   bts(Operand(object, Page::kDirtyFlagOffset), addr)?

and get rid of two moves?

http://codereview.chromium.org/2101002/show

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

Reply via email to