http://codereview.chromium.org/141032/diff/1/2
File src/x64/assembler-x64.cc (right):

http://codereview.chromium.org/141032/diff/1/2#newcode1197
Line 1197: emit_rex_64(dst);
On 2009/06/22 07:52:46, Lasse Reichstein wrote:
> If REX.W isn't required, we might as well use
emit_optional_rex_32(dst)


It is misleading to the code reader to read rex_32 for an actual 64-bit
move.  Actually, REX without REX.W may be diffrerent than no REX at all.

http://codereview.chromium.org/141032

--~--~---------~--~----~------------~-------~--~----~
v8-dev mailing list
v8-dev@googlegroups.com
http://groups.google.com/group/v8-dev
-~----------~----~----~----~------~----~------~--~---

Reply via email to