Philippe Mathieu-Daudé <f4...@amsat.org> writes: > The CPU() macro is defined as: > > #define CPU(obj) ((CPUState *)(obj)) > > which expands to: > > ((CPUState *)object_dynamic_cast_assert((Object *)(obj), (name), > __FILE__, __LINE__, __func__)) > > This assertion can only fail when @obj points to something other > than its stated type, i.e. when we're in undefined behavior country. > > Remove the unnecessary CPU() casts when we already know the pointer > is of CPUState type. > > Patch created mechanically using spatch with this script: > > @@ > typedef CPUState; > CPUState *s; > @@ > - CPU(s) > + s > > Acked-by: David Gibson <da...@gibson.dropbear.id.au> > Reviewed-by: Cédric Le Goater <c...@kaod.org> > Reviewed-by: Richard Henderson <richard.hender...@linaro.org> > Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org>
Reviewed-by: Markus Armbruster <arm...@redhat.com>