This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
PATCH: Correct fidivr operand size
- From: "H.J. Lu" <hongjiu dot lu at intel dot com>
- To: binutils at sources dot redhat dot com
- Date: Wed, 27 Aug 2008 10:34:15 -0700
- Subject: PATCH: Correct fidivr operand size
- Reply-to: "H.J. Lu" <hjl dot tools at gmail dot com>
I am checking in this patch to correct fidivr operand size.
H.J.
---
gas/testsuite/
2008-08-27 H.J. Lu <hongjiu.lu@intel.com>
* gas/i386/intel.s: Add tests for fidivr.
* gas/i386/intel.d: Updated.
opcodes/
2008-08-27 H.J. Lu <hongjiu.lu@intel.com>
* i386-opc.tbl: Correct fidivr operand size.
* i386-tbl.h: Regenerated.
--- binutils/gas/testsuite/gas/i386/intel.d.foo 2008-01-08 10:38:09.000000000 -0800
+++ binutils/gas/testsuite/gas/i386/intel.d 2008-08-27 10:21:27.000000000 -0700
@@ -688,4 +688,6 @@ Disassembly of section .text:
[ ]*[a-f0-9]+: de e3 fsubp %st,%st\(3\)
[ ]*[a-f0-9]+: de e3 fsubp %st,%st\(3\)
[ ]*[a-f0-9]+: de e3 fsubp %st,%st\(3\)
+[ ]*[a-f0-9]+: de 3b fidivr \(%ebx\)
+[ ]*[a-f0-9]+: da 3b fidivrl \(%ebx\)
#pass
--- binutils/gas/testsuite/gas/i386/intel.s.foo 2008-01-13 09:04:04.000000000 -0800
+++ binutils/gas/testsuite/gas/i386/intel.s 2008-08-27 10:17:56.000000000 -0700
@@ -686,3 +686,6 @@ fsubrp
fsubrp st(3)
fsubrp st(3),st
fsubrp st,st(3)
+
+fidivr word ptr [ebx]
+fidivr dword ptr [ebx]
--- binutils/opcodes/i386-opc.tbl.foo 2008-08-22 14:21:41.000000000 -0700
+++ binutils/opcodes/i386-opc.tbl 2008-08-27 09:59:21.000000000 -0700
@@ -698,7 +698,7 @@ fdivr, 0, 0xdef9, None, 2, 0, No_bSuf|No
fdivr, 0, 0xdef1, None, 2, 0, No_bSuf|No_wSuf|No_lSuf|No_sSuf|No_qSuf|No_ldSuf|Ugh|ATTMnemonic, { 0 }
fdivr, 2, 0xd8f8, None, 2, 0, No_bSuf|No_wSuf|No_lSuf|No_sSuf|No_ldSuf|No_qSuf|ShortForm|FloatD|FloatR, { FloatReg, FloatAcc }
fdivr, 1, 0xd8, 0x7, 1, 0, Modrm|FloatMF|No_bSuf|No_wSuf|No_qSuf|No_ldSuf, { Dword|Qword|Unspecified|BaseIndex|Disp8|Disp16|Disp32|Disp32S }
-fidivr, 1, 0xde, 0x7, 1, 0, Modrm|FloatMF|No_bSuf|No_wSuf|No_qSuf|No_ldSuf, { Dword|Qword|Unspecified|BaseIndex|Disp8|Disp16|Disp32|Disp32S }
+fidivr, 1, 0xde, 0x7, 1, 0, Modrm|FloatMF|No_bSuf|No_wSuf|No_qSuf|No_ldSuf, { Word|Dword|Unspecified|BaseIndex|Disp8|Disp16|Disp32|Disp32S }
fdivrp, 2, 0xdef8, None, 2, 0, ShortForm|No_bSuf|No_wSuf|No_lSuf|No_sSuf|No_qSuf|No_ldSuf|ATTMnemonic|ATTSyntax, { FloatAcc, FloatReg }
fdivrp, 1, 0xdef8, None, 2, 0, ShortForm|No_bSuf|No_wSuf|No_lSuf|No_sSuf|No_qSuf|No_ldSuf|ATTMnemonic|ATTSyntax, { FloatReg }
--- binutils/opcodes/i386-tbl.h.foo 2008-08-22 14:21:41.000000000 -0700
+++ binutils/opcodes/i386-tbl.h 2008-08-27 10:14:00.000000000 -0700
@@ -5120,7 +5120,7 @@ const template i386_optab[] =
1, 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0 },
{ { { 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
- 1, 1, 1, 1, 0, 0, 0, 1, 0, 0, 0, 0, 0, 0, 0, 0, 1, 0, 1, 0,
+ 1, 1, 1, 1, 0, 0, 0, 1, 0, 0, 0, 0, 0, 0, 0, 1, 1, 0, 0, 0,
0, 0, 1, 0, 0, 0 } } } },
{ "fdivrp", 2, 0xdef8, None, 2,
{ { 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,