diff options
| author | Richard Henderson <richard.henderson@linaro.org> | 2021-06-13 16:21:01 -0700 |
|---|---|---|
| committer | Richard Henderson <richard.henderson@linaro.org> | 2021-06-29 10:04:57 -0700 |
| commit | ebdd503d4572cc446a9a61410755cc3b87e2d76f (patch) | |
| tree | 6895e30a1a6e5ffb5ed68b17cf3714303e0d08cf /target | |
| parent | 50a7470e3e9de7fda510acd02880f85ad6d5afd5 (diff) | |
| download | qemu-ebdd503d4572cc446a9a61410755cc3b87e2d76f.zip qemu-ebdd503d4572cc446a9a61410755cc3b87e2d76f.tar.gz qemu-ebdd503d4572cc446a9a61410755cc3b87e2d76f.tar.bz2 | |
target/arm: Improve REVSH
The new bswap flags can implement the semantics exactly.
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
Diffstat (limited to 'target')
| -rw-r--r-- | target/arm/translate.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/target/arm/translate.c b/target/arm/translate.c index 669b0be..a0c6cfa 100644 --- a/target/arm/translate.c +++ b/target/arm/translate.c @@ -354,9 +354,7 @@ void gen_rev16(TCGv_i32 dest, TCGv_i32 var) /* Byteswap low halfword and sign extend. */ static void gen_revsh(TCGv_i32 dest, TCGv_i32 var) { - tcg_gen_ext16u_i32(var, var); - tcg_gen_bswap16_i32(var, var, TCG_BSWAP_IZ | TCG_BSWAP_OZ); - tcg_gen_ext16s_i32(dest, var); + tcg_gen_bswap16_i32(var, var, TCG_BSWAP_OS); } /* Dual 16-bit add. Result placed in t0 and t1 is marked as dead. |