diff options
| author | Richard Henderson <richard.henderson@linaro.org> | 2022-04-30 22:49:48 -0700 |
|---|---|---|
| committer | Peter Maydell <peter.maydell@linaro.org> | 2022-05-05 09:35:51 +0100 |
| commit | d385a60571487d9409f247d07aac94d1ee40f9fe (patch) | |
| tree | bac12f154f3f1d0a37618264a65e3c81e29927a4 /target/arm/helper.c | |
| parent | 87c3f0f2f791094adce2b8643d9103dac1b59702 (diff) | |
| download | focaccia-qemu-d385a60571487d9409f247d07aac94d1ee40f9fe.tar.gz focaccia-qemu-d385a60571487d9409f247d07aac94d1ee40f9fe.zip | |
target/arm: Avoid bare abort() or assert(0)
Standardize on g_assert_not_reached() for "should not happen". Retain abort() when preceeded by fprintf or error_report. Signed-off-by: Richard Henderson <richard.henderson@linaro.org> Reviewed-by: Peter Maydell <peter.maydell@linaro.org> Message-id: 20220501055028.646596-7-richard.henderson@linaro.org Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Diffstat (limited to 'target/arm/helper.c')
| -rw-r--r-- | target/arm/helper.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/target/arm/helper.c b/target/arm/helper.c index f84377babe..06f8864c77 100644 --- a/target/arm/helper.c +++ b/target/arm/helper.c @@ -8740,8 +8740,7 @@ void define_one_arm_cp_reg_with_opaque(ARMCPU *cpu, break; default: /* broken reginfo with out-of-range opc1 */ - assert(false); - break; + g_assert_not_reached(); } /* assert our permissions are not too lax (stricter is fine) */ assert((r->access & ~mask) == 0); @@ -10823,7 +10822,7 @@ static bool get_phys_addr_v5(CPUARMState *env, uint32_t address, break; default: /* Never happens, but compiler isn't smart enough to tell. */ - abort(); + g_assert_not_reached(); } } *prot = ap_to_rw_prot(env, mmu_idx, ap, domain_prot); @@ -10944,7 +10943,7 @@ static bool get_phys_addr_v6(CPUARMState *env, uint32_t address, break; default: /* Never happens, but compiler isn't smart enough to tell. */ - abort(); + g_assert_not_reached(); } } if (domain_prot == 3) { |