Re: [Qemu-devel] [PATCH 11/67] target/arm: Add stubs for aa32 decodetree

[Richard Henderson]

On 8/8/19 4:41 AM, Aleksandar Markovic wrote:
>     +/*
>     + * Include the generated decoders.
>     + * Note that the T32 decoder reuses some of the trans_* functions
>     + * initially declared by the A32 decoder, which results in duplicate
>     + * declaration warnings.  Suppress them.
>     + */
>     +
>     +#ifdef CONFIG_PRAGMA_DIAGNOSTIC_AVAILABLE
>     +# pragma GCC diagnostic push
>     +# pragma GCC diagnostic ignored "-Wredundant-decls"
>     +# ifdef __clang__
>     +#  pragma GCC diagnostic ignored "-Wtypedef-redefinition"
>     +# endif
>     +#endif
>     +
> 
> 
> This looks more like a "band aid" solution rather than the right one.

What would the "right" solution be, would you say?

A couple of days ago Phil suggested moving these pragmas into the generated
code, so that this need not be done by hand in the several targets that use
multiple decoders.  That sounds reasonable to me.


r~