[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v3 06/34] target/ppc: replace assert(0) with g_assert_not_reached
From: |
Pierrick Bouvier |
Subject: |
[PATCH v3 06/34] target/ppc: replace assert(0) with g_assert_not_reached() |
Date: |
Wed, 18 Sep 2024 21:46:13 -0700 |
This patch is part of a series that moves towards a consistent use of
g_assert_not_reached() rather than an ad hoc mix of different
assertion mechanisms.
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Daniel Henrique Barboza <dbarboza@ventanamicro.com>
Signed-off-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
---
target/ppc/dfp_helper.c | 8 ++++----
target/ppc/mmu_helper.c | 2 +-
2 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/target/ppc/dfp_helper.c b/target/ppc/dfp_helper.c
index 5967ea07a92..ecc3f793267 100644
--- a/target/ppc/dfp_helper.c
+++ b/target/ppc/dfp_helper.c
@@ -249,7 +249,7 @@ static void dfp_set_FPRF_from_FRT_with_context(struct
PPC_DFP *dfp,
fprf = 0x05;
break;
default:
- assert(0); /* should never get here */
+ g_assert_not_reached();
}
dfp->env->fpscr &= ~FP_FPRF;
dfp->env->fpscr |= (fprf << FPSCR_FPRF);
@@ -1243,7 +1243,7 @@ void helper_##op(CPUPPCState *env, ppc_fprp_t *t,
ppc_fprp_t *b) \
} else if (decNumberIsQNaN(&dfp.b)) { \
vt.VsrD(1) = -2; \
} else { \
- assert(0); \
+ g_assert_not_reached(); \
} \
set_dfp64(t, &vt); \
} else { \
@@ -1252,7 +1252,7 @@ void helper_##op(CPUPPCState *env, ppc_fprp_t *t,
ppc_fprp_t *b) \
} else if ((size) == 128) { \
vt.VsrD(1) = dfp.b.exponent + 6176; \
} else { \
- assert(0); \
+ g_assert_not_reached(); \
} \
set_dfp64(t, &vt); \
} \
@@ -1300,7 +1300,7 @@ void helper_##op(CPUPPCState *env, ppc_fprp_t *t,
ppc_fprp_t *a, \
raw_inf = 0x1e000; \
bias = 6176; \
} else { \
- assert(0); \
+ g_assert_not_reached(); \
} \
\
if (unlikely((exp < 0) || (exp > max_exp))) { \
diff --git a/target/ppc/mmu_helper.c b/target/ppc/mmu_helper.c
index b0a0676beba..b167b37e0ab 100644
--- a/target/ppc/mmu_helper.c
+++ b/target/ppc/mmu_helper.c
@@ -316,7 +316,7 @@ void ppc_tlb_invalidate_one(CPUPPCState *env, target_ulong
addr)
break;
default:
/* Should never reach here with other MMU models */
- assert(0);
+ g_assert_not_reached();
}
#else
ppc_tlb_invalidate_all(env);
--
2.39.5
- [PATCH v3 00/34] Use g_assert_not_reached instead of (g_)assert(0, false), Pierrick Bouvier, 2024/09/19
- [PATCH v3 01/34] hw/acpi: replace assert(0) with g_assert_not_reached(), Pierrick Bouvier, 2024/09/19
- [PATCH v3 02/34] hw/arm: replace assert(0) with g_assert_not_reached(), Pierrick Bouvier, 2024/09/19
- [PATCH v3 04/34] migration: replace assert(0) with g_assert_not_reached(), Pierrick Bouvier, 2024/09/19
- [PATCH v3 03/34] hw/net: replace assert(0) with g_assert_not_reached(), Pierrick Bouvier, 2024/09/19
- [PATCH v3 05/34] qobject: replace assert(0) with g_assert_not_reached(), Pierrick Bouvier, 2024/09/19
- [PATCH v3 06/34] target/ppc: replace assert(0) with g_assert_not_reached(),
Pierrick Bouvier <=
- [PATCH v3 07/34] block: replace assert(false) with g_assert_not_reached(), Pierrick Bouvier, 2024/09/19
- [PATCH v3 09/34] hw/net: replace assert(false) with g_assert_not_reached(), Pierrick Bouvier, 2024/09/19
- [PATCH v3 08/34] hw/hyperv: replace assert(false) with g_assert_not_reached(), Pierrick Bouvier, 2024/09/19
- [PATCH v3 10/34] hw/nvme: replace assert(false) with g_assert_not_reached(), Pierrick Bouvier, 2024/09/19
- [PATCH v3 11/34] hw/pci: replace assert(false) with g_assert_not_reached(), Pierrick Bouvier, 2024/09/19
- [PATCH v3 12/34] hw/ppc: replace assert(false) with g_assert_not_reached(), Pierrick Bouvier, 2024/09/19
- [PATCH v3 13/34] migration: replace assert(false) with g_assert_not_reached(), Pierrick Bouvier, 2024/09/19
- [PATCH v3 14/34] target/i386/kvm: replace assert(false) with g_assert_not_reached(), Pierrick Bouvier, 2024/09/19
- [PATCH v3 15/34] accel/tcg: remove break after g_assert_not_reached(), Pierrick Bouvier, 2024/09/19
- [PATCH v3 17/34] hw/acpi: remove break after g_assert_not_reached(), Pierrick Bouvier, 2024/09/19