On Thu, Apr 25, 2019 at 10:28 AM Richard Henderson <richard.hender...@linaro.org> wrote: > > --- > target/riscv/translate.c | 13 ++++++++++--- > 1 file changed, 10 insertions(+), 3 deletions(-)
Not sure how this will work with a fixup, but still: Reviewed-by: Alistair Francis <alistair.fran...@wdc.com> Alistair > > diff --git a/target/riscv/translate.c b/target/riscv/translate.c > index d1f599a92d..009c146e8f 100644 > --- a/target/riscv/translate.c > +++ b/target/riscv/translate.c > @@ -713,12 +713,19 @@ static bool gen_shift(DisasContext *ctx, arg_r *a, > * initially declared by the 32-bit decoder, which results in duplicate > * declaration warnings. Suppress them. > */ > -#pragma GCC diagnostic push > -#pragma GCC diagnostic ignored "-Wredundant-decls" > +#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 > > #include "decode_insn16.inc.c" > > -#pragma GCC diagnostic pop > +#ifdef CONFIG_PRAGMA_DIAGNOSTIC_AVAILABLE > +# pragma GCC diagnostic pop > +#endif > > static void decode_opc(DisasContext *ctx) > { > -- > 2.17.1 > >