Re: [PATCH v4 01/33] target/alpha: Tidy up alpha_cpu_class_by_name()

[Gavin Shan]

On 11/2/23 13:43, Richard Henderson wrote:
> On 11/1/23 17:24, Gavin Shan wrote:
> > From: Philippe Mathieu-Daudé <philmd@linaro.org>
> >
> > For target/alpha, the default CPU model name is "ev67". The default
> > CPU model is used when no matching CPU model is found. The conditions
> > to fall back to the default CPU model can be combined so that the code
> > looks a bit simplified.
> >
> > Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> > Reviewed-by: Gavin Shan <gshan@redhat.com>
> > ---
> >   target/alpha/cpu.c | 7 ++-----
> >   1 file changed, 2 insertions(+), 5 deletions(-)
> >
> > diff --git a/target/alpha/cpu.c b/target/alpha/cpu.c
> > index 51b7d8d1bf..c7ae4d6a41 100644
> > --- a/target/alpha/cpu.c
> > +++ b/target/alpha/cpu.c
> > @@ -142,13 +142,10 @@ static ObjectClass *alpha_cpu_class_by_name(const char 
> > *cpu_model)
> >       typename = g_strdup_printf(ALPHA_CPU_TYPE_NAME("%s"), cpu_model);
> >       oc = object_class_by_name(typename);
> >       g_free(typename);
> > -    if (oc != NULL && object_class_is_abstract(oc)) {
> > -        oc = NULL;
> > -    }
> >       /* TODO: remove match everything nonsense */
> > -    /* Default to ev67; no reason not to emulate insns by default. */
> > -    if (!oc) {
> > +    if (!oc || object_class_is_abstract(oc)) {
> > +        /* Default to ev67, no reason not to emulate insns by default */
> >           oc = object_class_by_name(ALPHA_CPU_TYPE_NAME("ev67"));
> >       }
>
> Acked-by: Richard Henderson <richard.henderson@linaro.org>
>
> I assume this TODO gets cleaned up later?

Let me drop this TODO in next respin.

Thanks,
Gavin