[igt-dev] [PATCH i-g-t] assembler: Fix or suppress clang warnings

Petri Latvala petri.latvala at intel.com
Mon Feb 1 07:28:25 UTC 2021


On Sun, Jan 31, 2021 at 10:38:04AM +0000, Chris Wilson wrote:
> For the simple assigment of constant strings to a char*, fix the type.
> 
> For the autogenerated bison output, tell gcc/clang to ignore the
> warnings.
> 
> v2: When in doubt disable the warning generated by the bison output
> 
> Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>

Reviewed-by: Petri Latvala <petri.latvala at intel.com>


> ---
>  assembler/brw_context.h | 2 +-
>  assembler/gen4asm.h     | 2 +-
>  assembler/gen8_disasm.c | 2 ++
>  assembler/gram.y        | 2 ++
>  assembler/lex.l         | 2 +-
>  assembler/main.c        | 2 +-
>  assembler/meson.build   | 1 +
>  assembler/ralloc.c      | 2 ++
>  8 files changed, 11 insertions(+), 4 deletions(-)
> 
> diff --git a/assembler/brw_context.h b/assembler/brw_context.h
> index 90e66f714..335234e37 100644
> --- a/assembler/brw_context.h
> +++ b/assembler/brw_context.h
> @@ -62,7 +62,7 @@ brw_init_context(struct brw_context *brw, int gen);
>  
>  /* brw_disasm.c */
>  struct opcode_desc {
> -    char    *name;
> +    const char    *name;
>      int	    nsrc;
>      int	    ndst;
>  };
> diff --git a/assembler/gen4asm.h b/assembler/gen4asm.h
> index 9537d4573..b1680ebc7 100644
> --- a/assembler/gen4asm.h
> +++ b/assembler/gen4asm.h
> @@ -46,7 +46,7 @@ extern int errors;
>  #define WARN_ALL	(1 << 31)
>  extern unsigned int warning_flags;
>  
> -extern char *input_filename;
> +extern const char *input_filename;
>  
>  extern struct brw_context genasm_context;
>  extern struct brw_compile genasm_compile;
> diff --git a/assembler/gen8_disasm.c b/assembler/gen8_disasm.c
> index 68bc03347..99b062ea3 100644
> --- a/assembler/gen8_disasm.c
> +++ b/assembler/gen8_disasm.c
> @@ -31,6 +31,8 @@
>  #include "brw_defines.h"
>  #include "gen8_instruction.h"
>  
> +#pragma GCC diagnostic ignored "-Wformat-nonliteral"
> +
>  static const struct opcode_desc *m_opcode = opcode_descs;
>  
>  static const char *const m_conditional_modifier[16] = {
> diff --git a/assembler/gram.y b/assembler/gram.y
> index d6f4a195b..77975e072 100644
> --- a/assembler/gram.y
> +++ b/assembler/gram.y
> @@ -36,6 +36,8 @@
>  #include "brw_eu.h"
>  #include "gen8_instruction.h"
>  
> +#pragma GCC diagnostic ignored "-Wformat-nonliteral"
> +
>  #define DEFAULT_EXECSIZE (ffs(program_defaults.execute_size) - 1)
>  #define DEFAULT_DSTREGION -1
>  
> diff --git a/assembler/lex.l b/assembler/lex.l
> index 1ba576bfe..c12c5054c 100644
> --- a/assembler/lex.l
> +++ b/assembler/lex.l
> @@ -7,7 +7,7 @@
>  
>  #include "string.h"
>  int saved_state = 0;
> -extern char *input_filename;
> +extern const char *input_filename;
>  
>  /* Locations */
>  int yycolumn = 1;
> diff --git a/assembler/main.c b/assembler/main.c
> index a1eca5251..2d39d4536 100644
> --- a/assembler/main.c
> +++ b/assembler/main.c
> @@ -45,7 +45,7 @@ long int gen_level = 40;
>  int advanced_flag = 0; /* 0: in unit of byte, 1: in unit of data element size */
>  unsigned int warning_flags = WARN_ALWAYS;
>  int need_export = 0;
> -char *input_filename = "<stdin>";
> +const char *input_filename = "<stdin>";
>  int errors;
>  
>  struct brw_context genasm_brw_context;
> diff --git a/assembler/meson.build b/assembler/meson.build
> index 3b4af348a..68e6a9209 100644
> --- a/assembler/meson.build
> +++ b/assembler/meson.build
> @@ -18,6 +18,7 @@ assembler_warnflags = [
>  	'-Wno-old-style-definition',
>  	'-Wno-redundant-decls',
>  	'-Wno-shadow',
> +	'-Wno-incompatible-pointer-types-discards-qualifiers',
>  ]
>  assembler_args = []
>  foreach flag : assembler_warnflags
> diff --git a/assembler/ralloc.c b/assembler/ralloc.c
> index 69c1da4d3..772c34448 100644
> --- a/assembler/ralloc.c
> +++ b/assembler/ralloc.c
> @@ -36,6 +36,8 @@ _CRTIMP int _vscprintf(const char *format, va_list argptr);
>  
>  #include "ralloc.h"
>  
> +#pragma GCC diagnostic ignored "-Wformat-nonliteral"
> +
>  #ifndef va_copy
>  #ifdef __va_copy
>  #define va_copy(dest, src) __va_copy((dest), (src))
> -- 
> 2.30.0
> 


More information about the igt-dev mailing list