diff options
| author | Mike Frysinger <vapier@gentoo.org> | 2011-06-18 19:42:55 +0000 |
|---|---|---|
| committer | Mike Frysinger <vapier@gentoo.org> | 2011-06-18 19:42:55 +0000 |
| commit | ef26d60eba5736240c27c545e275667bb4b0fbd1 (patch) | |
| tree | 1964dea48acfb2df2639d14cb5cb8fd8b8b62f5b /include/opcode/bfin.h | |
| parent | 075a845a9281b4bdef459b6494a7db3bf666b911 (diff) | |
| download | gdb-ef26d60eba5736240c27c545e275667bb4b0fbd1.zip gdb-ef26d60eba5736240c27c545e275667bb4b0fbd1.tar.gz gdb-ef26d60eba5736240c27c545e275667bb4b0fbd1.tar.bz2 | |
sim: bfin: fix sign extension in dsp insns with MM flag
After testing the hardware with all the different dsp flags, the MM flag
triggers sign extension in all modes. So drop the limited use of it, and
the local custom helper that was also extending unsigned values. We also
can see that the flag checks in the mult/mac insns have the same behavior
with sign extending, so add a helper func to keep the logic the same in
both places.
Signed-off-by: Robin Getz <robin.getz@analog.com>
Signed-off-by: Mike Frysinger <vapier@gentoo.org>
Diffstat (limited to 'include/opcode/bfin.h')
| -rwxr-xr-x | include/opcode/bfin.h | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/include/opcode/bfin.h b/include/opcode/bfin.h index 30bb90d..26f0193 100755 --- a/include/opcode/bfin.h +++ b/include/opcode/bfin.h @@ -53,6 +53,12 @@ static inline int is_macmod_hmove (int x) || (x == M_TFU) || (x == M_S2RND) || (x == M_ISS2) || (x == M_IH); } +static inline int is_macmod_signed (int x) +{ + return (x == 0) || (x == M_IS) || (x == M_T) || (x == M_S2RND) + || (x == M_ISS2) || (x == M_IH) || (x == M_W32); +} + /* dsp32mac +----+----+---+---|---+----+----+---|---+---+---+---|---+---+---+---+ | 1 | 1 | 0 | 0 |.M.| 0 | 0 |.mmod..........|.MM|.P.|.w1|.op1...| |