diff options
| author | Víctor Colombo | 2022-10-05 14:15:51 +0200 |
|---|---|---|
| committer | Daniel Henrique Barboza | 2022-10-17 21:15:09 +0200 |
| commit | c348e09fb077553469f3d9be41ba671e63733a9b (patch) | |
| tree | 279a500bcc871cfdb13cac090b4e6c6b0e248ebb /target/ppc/translate | |
| parent | hw/ppc/e500: Remove if statement which is now always true (diff) | |
| download | qemu-c348e09fb077553469f3d9be41ba671e63733a9b.tar.gz qemu-c348e09fb077553469f3d9be41ba671e63733a9b.tar.xz qemu-c348e09fb077553469f3d9be41ba671e63733a9b.zip | |
target/ppc: Fix xvcmp* clearing FI bit
Vector instructions in general are not supposed to change the FI bit.
However, xvcmp* instructions are calling gen_helper_float_check_status,
which is leading to a cleared FI flag where it should be kept
unchanged.
As helper_float_check_status only affects inexact, overflow and
underflow, and the xvcmp* instructions don't change these flags, this
issue can be fixed by removing the call to helper_float_check_status.
By doing this, the FI bit in FPSCR will be preserved as expected.
Fixes: 00084a25adf ("target/ppc: introduce separate VSX_CMP macro for xvcmp* instructions")
Signed-off-by: Víctor Colombo <victor.colombo@eldorado.org.br>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Message-Id: <20221005121551.27957-1-victor.colombo@eldorado.org.br>
Signed-off-by: Daniel Henrique Barboza <danielhb413@gmail.com>
Diffstat (limited to 'target/ppc/translate')
| -rw-r--r-- | target/ppc/translate/vsx-impl.c.inc | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/target/ppc/translate/vsx-impl.c.inc b/target/ppc/translate/vsx-impl.c.inc index 7acdbceec4..e6e5c45ffd 100644 --- a/target/ppc/translate/vsx-impl.c.inc +++ b/target/ppc/translate/vsx-impl.c.inc @@ -810,7 +810,6 @@ static void gen_##name(DisasContext *ctx) \ gen_helper_##name(ignored, cpu_env, xt, xa, xb); \ tcg_temp_free_i32(ignored); \ } \ - gen_helper_float_check_status(cpu_env); \ tcg_temp_free_ptr(xt); \ tcg_temp_free_ptr(xa); \ tcg_temp_free_ptr(xb); \ |