Toolchain/binutils-gdb
1
0
Fork 0
forked from Imagelibrary/binutils-gdb
Code Pull Requests Activity

sim: sh: adjust some dsp insn masks

Browse Source 
The pmuls encoding is incorrect -- it looks like a copy & paste error
from the padd pmuls variant.  The SuperH software manual covers this.

On the flip side, the manual lists pwsb & pwad as insns that exist,
but no description of what they do, what the insn name means, or the
actual encoding.  Our sim implementation stubs them both out as nops.
Let's mark the fields to avoid unused variable warnings.
This commit is contained in:
Mike Frysinger
2023-12-15 23:54:40 -05:00
parent 0fd9d0cec0
commit 82a398adb8
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
sim/sh/gencode.c
View File

@@ -1915,7 +1915,7 @@ static op ppi_tab[] =
"COMPUTE_OVERFLOW;",
"greater_equal = 0;",
},
{ "","", "pmuls Se,Sf,Dg", "0100eeffxxyygguu",
{ "","", "pmuls Se,Sf,Dg", "0100eeff0000gg00",
"res = (DSP_R (e) >> 16) * (DSP_R (f) >> 16) * 2;",
"if (res == 0x80000000)",
" res = 0x7fffffff;",
@@ -2011,9 +2011,9 @@ static op ppi_tab[] =
"COMPUTE_OVERFLOW;",
"ADD_SUB_GE;",
},
{ "","", "pwsb Sx,Sy,Dz", "10100100xxyyzzzz",
{ "","", "pwsb Sx,Sy,Dz", "10100100....zzzz",
},
{ "","", "pwad Sx,Sy,Dz", "10110100xxyyzzzz",
{ "","", "pwad Sx,Sy,Dz", "10110100....zzzz",
},
{ "","", "(if cc) pabs Sx,Dz", "100010ccxx01zzzz",
"/* FIXME: duplicate code pabs. */",