diff options
author | Chris Wilson <chris@chris-wilson.co.uk> | 2021-01-31 10:22:47 +0000 |
---|---|---|
committer | Chris Wilson <chris@chris-wilson.co.uk> | 2021-02-01 15:29:40 +0000 |
commit | ccfc101419c1e233f87ca509781c5f8e09a2f3ae (patch) | |
tree | f56bb91312474ec8bf430e6230c2e84d4143575f /assembler | |
parent | dda93ce17e3d0e025b5e9aa83271438cc41ff734 (diff) |
assembler: Fix or suppress clang warnings
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@chris-wilson.co.uk>
Reviewed-by: Petri Latvala <petri.latvala@intel.com>
Diffstat (limited to 'assembler')
-rw-r--r-- | assembler/brw_context.h | 2 | ||||
-rw-r--r-- | assembler/gen4asm.h | 2 | ||||
-rw-r--r-- | assembler/gen8_disasm.c | 2 | ||||
-rw-r--r-- | assembler/gram.y | 2 | ||||
-rw-r--r-- | assembler/lex.l | 2 | ||||
-rw-r--r-- | assembler/main.c | 2 | ||||
-rw-r--r-- | assembler/meson.build | 1 | ||||
-rw-r--r-- | assembler/ralloc.c | 2 |
8 files changed, 11 insertions, 4 deletions
diff --git a/assembler/brw_context.h b/assembler/brw_context.h index 90e66f71..335234e3 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 9537d457..b1680ebc 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 68bc0334..99b062ea 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 d6f4a195..77975e07 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 1ba576bf..c12c5054 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 a1eca525..2d39d453 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 3b4af348..68e6a920 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 69c1da4d..772c3444 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)) |