* [9fans] MOVLQSX wrong operand order @ 2014-01-17 20:07 cinap_lenrek 2014-01-17 20:40 ` erik quanstrom 2014-01-17 21:07 ` Steven Stallion 0 siblings, 2 replies; 3+ messages in thread From: cinap_lenrek @ 2014-01-17 20:07 UTC (permalink / raw) To: quanstro; +Cc: 9fans just disassembled some code generated by 6c and found something funny. the debugger seems to print the operands of MOVLQSX in the wrong order. output of 6c -S: MOVLQSX BP,CX ADDQ CX,AX RET , output of acid: foo+0x1a 0x00200042 MOVLQSX CX,BP foo+0x1d 0x00200045 ADDQ CX,AX foo+0x20 0x00200048 RET fix: diff -r 4e094627d459 sys/src/libmach/8db.c --- a/sys/src/libmach/8db.c Tue Jan 14 00:22:13 2014 +0100 +++ b/sys/src/libmach/8db.c Fri Jan 17 21:06:42 2014 +0100 @@ -1223,7 +1223,7 @@ [0xfd] 0,0, "STD", [0xfe] RMOPB,0, optabFE, [0xff] RMOP,0, optabFF, -[0x100] RM,0, "MOVLQSX %r,%e", +[0x100] RM,0, "MOVLQSX %e,%r", }; -- cinap ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [9fans] MOVLQSX wrong operand order 2014-01-17 20:07 [9fans] MOVLQSX wrong operand order cinap_lenrek @ 2014-01-17 20:40 ` erik quanstrom 2014-01-17 21:07 ` Steven Stallion 1 sibling, 0 replies; 3+ messages in thread From: erik quanstrom @ 2014-01-17 20:40 UTC (permalink / raw) To: 9fans On Fri Jan 17 15:09:33 EST 2014, cinap_lenrek@felloff.net wrote: > just disassembled some code generated by 6c > and found something funny. the debugger seems > to print the operands of MOVLQSX in the wrong > order. there may be more of these. maybe if , were replaced with → .... - erik ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [9fans] MOVLQSX wrong operand order 2014-01-17 20:07 [9fans] MOVLQSX wrong operand order cinap_lenrek 2014-01-17 20:40 ` erik quanstrom @ 2014-01-17 21:07 ` Steven Stallion 1 sibling, 0 replies; 3+ messages in thread From: Steven Stallion @ 2014-01-17 21:07 UTC (permalink / raw) To: Fans of the OS Plan 9 from Bell Labs On Fri, Jan 17, 2014 at 12:07 PM, <cinap_lenrek@felloff.net> wrote: > just disassembled some code generated by 6c > and found something funny. the debugger seems > to print the operands of MOVLQSX in the wrong > order. *db.c are notoriously out of date an almost all platforms. I spent a good chunk of time fixing up ARM (and adding a ton of ARMv7 stuff) last year. The patch is still sitting in patch/arm-refresh. At least my local copy is sane... Steve ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2014-01-17 21:07 UTC | newest] Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2014-01-17 20:07 [9fans] MOVLQSX wrong operand order cinap_lenrek 2014-01-17 20:40 ` erik quanstrom 2014-01-17 21:07 ` Steven Stallion
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).