From: Florian Krohm <flo2030@eich-krohm.de>
Date: Sun, 20 Jul 2025 17:58:10 +0000 (+0000)
Subject: Fix compiler warnings on ppc due to unused s390 specific variables.
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=c430ffabb27cc6f4b18a6ad1a71185e0e47b3c6a;p=thirdparty%2Fvalgrind.git

Fix compiler warnings on ppc due to unused s390 specific variables.
---

diff --git a/coregrind/m_debuginfo/debuginfo.c b/coregrind/m_debuginfo/debuginfo.c
index 4ce976a6d..63ed32608 100644
--- a/coregrind/m_debuginfo/debuginfo.c
+++ b/coregrind/m_debuginfo/debuginfo.c
@@ -3554,7 +3554,6 @@ Bool VG_(use_CF_info) ( /*MOD*/D3UnwindRegs* uregsHere,
 #  endif
 
 #  if defined(VGA_s390x)
-   const Bool is_s390x = True;
    const Addr old_S390X_F0 = uregsHere->f0;
    const Addr old_S390X_F1 = uregsHere->f1;
    const Addr old_S390X_F2 = uregsHere->f2;
@@ -3563,16 +3562,9 @@ Bool VG_(use_CF_info) ( /*MOD*/D3UnwindRegs* uregsHere,
    const Addr old_S390X_F5 = uregsHere->f5;
    const Addr old_S390X_F6 = uregsHere->f6;
    const Addr old_S390X_F7 = uregsHere->f7;
+#   define SAVE_TO_PREV(p,reg) do { p = reg; } while (0)
 #  else
-   const Bool is_s390x = False;
-   const Addr old_S390X_F0 = 0;
-   const Addr old_S390X_F1 = 0;
-   const Addr old_S390X_F2 = 0;
-   const Addr old_S390X_F3 = 0;
-   const Addr old_S390X_F4 = 0;
-   const Addr old_S390X_F5 = 0;
-   const Addr old_S390X_F6 = 0;
-   const Addr old_S390X_F7 = 0;
+#   define SAVE_TO_PREV(p,reg) do { vg_assert(0); } while (0)
 #  endif
 
 #  define COMPUTE(_prev, _here, _how, _off)             \
@@ -3604,29 +3596,29 @@ Bool VG_(use_CF_info) ( /*MOD*/D3UnwindRegs* uregsHere,
                if (!ok) return False;                   \
                break;                                   \
             case CFIR_S390X_F0:                               \
-               if (is_s390x) { _prev = old_S390X_F0; break; } \
-               vg_assert(0+0-0);                              \
+               SAVE_TO_PREV(_prev, old_S390X_F0);             \
+               break;                                         \
             case CFIR_S390X_F1:                               \
-               if (is_s390x) { _prev = old_S390X_F1; break; } \
-               vg_assert(0+1-1);                              \
+               SAVE_TO_PREV(_prev, old_S390X_F1);             \
+               break;                                         \
             case CFIR_S390X_F2:                               \
-               if (is_s390x) { _prev = old_S390X_F2; break; } \
-               vg_assert(0+2-2);                              \
+               SAVE_TO_PREV(_prev, old_S390X_F2);             \
+               break;                                         \
             case CFIR_S390X_F3:                               \
-               if (is_s390x) { _prev = old_S390X_F3; break; } \
-               vg_assert(0+3-3);                              \
+               SAVE_TO_PREV(_prev, old_S390X_F3);             \
+               break;                                         \
             case CFIR_S390X_F4:                               \
-               if (is_s390x) { _prev = old_S390X_F4; break; } \
-               vg_assert(0+4-4);                              \
+               SAVE_TO_PREV(_prev, old_S390X_F4);             \
+               break;                                         \
             case CFIR_S390X_F5:                               \
-               if (is_s390x) { _prev = old_S390X_F5; break; } \
-               vg_assert(0+5-5);                              \
+               SAVE_TO_PREV(_prev, old_S390X_F5);             \
+               break;                                         \
             case CFIR_S390X_F6:                               \
-               if (is_s390x) { _prev = old_S390X_F6; break; } \
-               vg_assert(0+6-6);                              \
+               SAVE_TO_PREV(_prev, old_S390X_F6);             \
+               break;                                         \
             case CFIR_S390X_F7:                               \
-               if (is_s390x) { _prev = old_S390X_F7; break; } \
-               vg_assert(0+7-7);                              \
+               SAVE_TO_PREV(_prev, old_S390X_F7);             \
+               break;                                         \
             default:                                    \
                vg_assert(0*0);                          \
          }                                              \