Modify 'movq reg, reg' encoding in 64-bit code.

Modify 'movq reg, reg' encoding in 64-bit code.
Use 0x89 encoding (instead of 0x8B encoding), which is expected by gdb64
older than 7.3.1-gg5 when disassembling a function's prolog (movq rbp, rsp)
for proper unwinding of Dart frames (use --generate_gdb_symbols and -O0).
Committed: https://code.google.com/p/dart/source/detail?r=6422

[hausner, 2012-04-11 18:04:09]

lgtm

[Lasse Reichstein Nielsen, 2012-04-12 09:57:38]

DBC.

https://chromiumcodereview.appspot.com/10052013/diff/1/runtime/vm/assembler_x...
File runtime/vm/assembler_x64.cc (right):

https://chromiumcodereview.appspot.com/10052013/diff/1/runtime/vm/assembler_x...
runtime/vm/assembler_x64.cc:288: EmitUint8(0x89);
You could decide whether to use 0x89 or 0x8b depending on whether src or dst is
rsp (or r12), i.e., use the encoding that doesn't require an extra SIP byte if
possible. That would give you 0x89 for 'movq rbp, rsp' and also save a byte for
'mov rsp, rbp'.
(The same trick can be done for most operations that have both r/m,r and r,r/m
addressing modes).

[regis, 2012-04-12 16:46:16]

https://chromiumcodereview.appspot.com/10052013/diff/1/runtime/vm/assembler_x...
File runtime/vm/assembler_x64.cc (right):

https://chromiumcodereview.appspot.com/10052013/diff/1/runtime/vm/assembler_x...
runtime/vm/assembler_x64.cc:288: EmitUint8(0x89);
On 2012/04/12 09:57:38, Lasse Reichstein Nielsen wrote:
> You could decide whether to use 0x89 or 0x8b depending on whether src or dst
is
> rsp (or r12), i.e., use the encoding that doesn't require an extra SIP byte if
> possible. That would give you 0x89 for 'movq rbp, rsp' and also save a byte
for
> 'mov rsp, rbp'.
> (The same trick can be done for most operations that have both r/m,r and r,r/m
> addressing modes).

This is a good point for memory addressing modes, but I do not think we ever
need a SIB byte in movq reg, reg.

[Lasse Reichstein Nielsen, 2012-04-13 08:33:38]

On 2012/04/12 16:46:16, regis wrote:
> This is a good point for memory addressing modes, but I do not think we ever
> need a SIB byte in movq reg, reg.

Ack, yes, that is true. It's only for memory operations. My bad.