Re: [PATCH 06/19] target/ppc: Set OV32 when OV is set

[Daniel Henrique Barboza]

On 9/1/22 10:17, Víctor Colombo wrote:

According to PowerISA: "OV32 is set whenever OV is implicitly set, and
is set to the same value that OV is defined to be set to in 32-bit
mode".

This patch changes helper_update_ov_legacy to set/clear ov32 when
applicable.

Signed-off-by: Víctor Colombo <victor.colo...@eldorado.org.br>
---

Reviewed-by: Daniel Henrique Barboza <danielhb...@gmail.com>

  target/ppc/int_helper.c | 4 ++--
  1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/target/ppc/int_helper.c b/target/ppc/int_helper.c
index d905f07d02..696096100b 100644
--- a/target/ppc/int_helper.c
+++ b/target/ppc/int_helper.c
@@ -37,9 +37,9 @@
  static inline void helper_update_ov_legacy(CPUPPCState *env, int ov)
  {
      if (unlikely(ov)) {
-        env->so = env->ov = 1;
+        env->so = env->ov = env->ov32 = 1;
      } else {
-        env->ov = 0;
+        env->ov = env->ov32 = 0;
      }
  }