[PATCH] target/m68k: Remove unused @cpu_exception_index TCGv

Philippe Mathieu-Daudé posted 1 patch 3 days, 21 hours ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20250924171308.53036-1-philmd@linaro.org
Maintainers: Laurent Vivier <laurent@vivier.eu>
target/m68k/translate.c | 6 ------
1 file changed, 6 deletions(-)
[PATCH] target/m68k: Remove unused @cpu_exception_index TCGv
Posted by Philippe Mathieu-Daudé 3 days, 21 hours ago
When moving the @exception_index from each target ArchCPU
to the global CPUState in commit 27103424c40 ("cpu: Move
exception_index field from CPU_COMMON to CPUState"), the
@cpu_exception_index TCGv has been created for m68k target.

For years, no code ever used this register. Simply remove it.

Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
---
 target/m68k/translate.c | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/target/m68k/translate.c b/target/m68k/translate.c
index 97afceb1297..0cee54f4900 100644
--- a/target/m68k/translate.c
+++ b/target/m68k/translate.c
@@ -45,7 +45,6 @@
 #undef DEFO64
 
 static TCGv_i32 cpu_halted;
-static TCGv_i32 cpu_exception_index;
 
 static char cpu_reg_names[2 * 8 * 3 + 5 * 4];
 static TCGv cpu_dregs[8];
@@ -81,11 +80,6 @@ void m68k_tcg_init(void)
     cpu_halted = tcg_global_mem_new_i32(tcg_env,
                                         -offsetof(M68kCPU, env) +
                                         offsetof(CPUState, halted), "HALTED");
-    cpu_exception_index = tcg_global_mem_new_i32(tcg_env,
-                                                 -offsetof(M68kCPU, env) +
-                                                 offsetof(CPUState, exception_index),
-                                                 "EXCEPTION");
-
     p = cpu_reg_names;
     for (i = 0; i < 8; i++) {
         sprintf(p, "D%d", i);
-- 
2.51.0


Re: [PATCH] target/m68k: Remove unused @cpu_exception_index TCGv
Posted by Richard Henderson 3 days, 21 hours ago
On 9/24/25 10:13, Philippe Mathieu-Daudé wrote:
> When moving the @exception_index from each target ArchCPU
> to the global CPUState in commit 27103424c40 ("cpu: Move
> exception_index field from CPU_COMMON to CPUState"), the
> @cpu_exception_index TCGv has been created for m68k target.
> 
> For years, no code ever used this register. Simply remove it.
> 
> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> ---
>   target/m68k/translate.c | 6 ------
>   1 file changed, 6 deletions(-)

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

r~

> 
> diff --git a/target/m68k/translate.c b/target/m68k/translate.c
> index 97afceb1297..0cee54f4900 100644
> --- a/target/m68k/translate.c
> +++ b/target/m68k/translate.c
> @@ -45,7 +45,6 @@
>   #undef DEFO64
>   
>   static TCGv_i32 cpu_halted;
> -static TCGv_i32 cpu_exception_index;
>   
>   static char cpu_reg_names[2 * 8 * 3 + 5 * 4];
>   static TCGv cpu_dregs[8];
> @@ -81,11 +80,6 @@ void m68k_tcg_init(void)
>       cpu_halted = tcg_global_mem_new_i32(tcg_env,
>                                           -offsetof(M68kCPU, env) +
>                                           offsetof(CPUState, halted), "HALTED");
> -    cpu_exception_index = tcg_global_mem_new_i32(tcg_env,
> -                                                 -offsetof(M68kCPU, env) +
> -                                                 offsetof(CPUState, exception_index),
> -                                                 "EXCEPTION");
> -
>       p = cpu_reg_names;
>       for (i = 0; i < 8; i++) {
>           sprintf(p, "D%d", i);