[Mesa-dev] [PATCH 10/13] intel/compiler: Silence unused parameter warnings
Caio Marcelo de Oliveira Filho
caio.oliveira at intel.com
Fri Aug 17 23:55:57 UTC 2018
You've fixed the warnings in "generate_*" functions in another patch
that landed. I guess this commit also fixed and the rebase was clean,
but the commit message remained.
With that update to the commit message, this patch is
Reviewed-by: Caio Marcelo de Oliveira Filho <caio.oliveira at intel.com>
On Fri, Jun 22, 2018 at 10:03:55PM -0700, Ian Romanick wrote:
> From: Ian Romanick <ian.d.romanick at intel.com>
>
> src/intel/compiler/brw_disasm_info.c: In function ‘nir_print_instr’:
> src/intel/compiler/brw_disasm_info.c:30:61: warning: unused parameter ‘instr’ [-Wunused-parameter]
> __attribute__((weak)) void nir_print_instr(const nir_instr *instr, FILE *fp) {}
> ^~~~~
> src/intel/compiler/brw_disasm_info.c:30:74: warning: unused parameter ‘fp’ [-Wunused-parameter]
> __attribute__((weak)) void nir_print_instr(const nir_instr *instr, FILE *fp) {}
> ^~
> src/intel/compiler/brw_disasm.c: In function ‘src_ia1’:
> src/intel/compiler/brw_disasm.c:850:18: warning: unused parameter ‘_reg_file’ [-Wunused-parameter]
> unsigned _reg_file,
> ^~~~~~~~~
> src/intel/compiler/brw_fs_surface_builder.cpp: In function ‘void brw::surface_access::emit_byte_scattered_write(const brw::fs_builder&, const fs_reg&, const fs_reg&, const fs_reg&, unsigned int, unsigned int, unsigned int, brw_predicate)’:
> src/intel/compiler/brw_fs_surface_builder.cpp:193:57: warning: unused parameter ‘size’ [-Wunused-parameter]
> unsigned dims, unsigned size,
> ^~~~
> src/intel/compiler/brw_fs_generator.cpp: In member function ‘void fs_generator::generate_barrier(fs_inst*, brw_reg)’:
> src/intel/compiler/brw_fs_generator.cpp:743:41: warning: unused parameter ‘inst’ [-Wunused-parameter]
> fs_generator::generate_barrier(fs_inst *inst, struct brw_reg src)
> ^~~~
> src/intel/compiler/brw_fs_generator.cpp: In member function ‘void fs_generator::generate_discard_jump(fs_inst*)’:
> src/intel/compiler/brw_fs_generator.cpp:1326:46: warning: unused parameter ‘inst’ [-Wunused-parameter]
> fs_generator::generate_discard_jump(fs_inst *inst)
> ^~~~
> src/intel/compiler/brw_fs_generator.cpp: In member function ‘void fs_generator::generate_pack_half_2x16_split(fs_inst*, brw_reg, brw_reg, brw_reg)’:
> src/intel/compiler/brw_fs_generator.cpp:1675:54: warning: unused parameter ‘inst’ [-Wunused-parameter]
> fs_generator::generate_pack_half_2x16_split(fs_inst *inst,
> ^~~~
> src/intel/compiler/brw_fs_generator.cpp: In member function ‘void fs_generator::generate_shader_time_add(fs_inst*, brw_reg, brw_reg, brw_reg)’:
> src/intel/compiler/brw_fs_generator.cpp:1743:49: warning: unused parameter ‘inst’ [-Wunused-parameter]
> fs_generator::generate_shader_time_add(fs_inst *inst,
> ^~~~
>
> Signed-off-by: Ian Romanick <ian.d.romanick at intel.com>
> ---
> src/intel/compiler/brw_disasm.c | 3 ---
> src/intel/compiler/brw_disasm_info.c | 3 ++-
> src/intel/compiler/brw_fs_nir.cpp | 2 +-
> src/intel/compiler/brw_fs_surface_builder.cpp | 2 +-
> src/intel/compiler/brw_fs_surface_builder.h | 2 +-
> 5 files changed, 5 insertions(+), 7 deletions(-)
>
> diff --git a/src/intel/compiler/brw_disasm.c b/src/intel/compiler/brw_disasm.c
> index 5f75c67942e..e45762afb07 100644
> --- a/src/intel/compiler/brw_disasm.c
> +++ b/src/intel/compiler/brw_disasm.c
> @@ -846,7 +846,6 @@ src_ia1(FILE *file,
> const struct gen_device_info *devinfo,
> unsigned opcode,
> enum brw_reg_type type,
> - unsigned _reg_file,
> int _addr_imm,
> unsigned _addr_subreg_nr,
> unsigned _negate,
> @@ -1326,7 +1325,6 @@ src0(FILE *file, const struct gen_device_info *devinfo, const brw_inst *inst)
> devinfo,
> brw_inst_opcode(devinfo, inst),
> brw_inst_src0_type(devinfo, inst),
> - brw_inst_src0_reg_file(devinfo, inst),
> brw_inst_src0_ia1_addr_imm(devinfo, inst),
> brw_inst_src0_ia_subreg_nr(devinfo, inst),
> brw_inst_src0_negate(devinfo, inst),
> @@ -1382,7 +1380,6 @@ src1(FILE *file, const struct gen_device_info *devinfo, const brw_inst *inst)
> devinfo,
> brw_inst_opcode(devinfo, inst),
> brw_inst_src1_type(devinfo, inst),
> - brw_inst_src1_reg_file(devinfo, inst),
> brw_inst_src1_ia1_addr_imm(devinfo, inst),
> brw_inst_src1_ia_subreg_nr(devinfo, inst),
> brw_inst_src1_negate(devinfo, inst),
> diff --git a/src/intel/compiler/brw_disasm_info.c b/src/intel/compiler/brw_disasm_info.c
> index f8f4c02e763..7a9a55d83ef 100644
> --- a/src/intel/compiler/brw_disasm_info.c
> +++ b/src/intel/compiler/brw_disasm_info.c
> @@ -27,7 +27,8 @@
> #include "common/gen_debug.h"
> #include "compiler/nir/nir.h"
>
> -__attribute__((weak)) void nir_print_instr(const nir_instr *instr, FILE *fp) {}
> +__attribute__((weak)) void nir_print_instr(UNUSED const nir_instr *instr,
> + UNUSED FILE *fp) {}
>
> void
> dump_assembly(void *assembly, struct disasm_info *disasm)
> diff --git a/src/intel/compiler/brw_fs_nir.cpp b/src/intel/compiler/brw_fs_nir.cpp
> index 0abb4798e70..243b1d2ae75 100644
> --- a/src/intel/compiler/brw_fs_nir.cpp
> +++ b/src/intel/compiler/brw_fs_nir.cpp
> @@ -4302,7 +4302,7 @@ fs_visitor::nir_emit_intrinsic(const fs_builder &bld, nir_intrinsic_instr *instr
> */
> emit_byte_scattered_write(bld, surf_index, offset_reg,
> write_src,
> - 1 /* dims */, 1,
> + 1 /* dims */,
> bit_size,
> BRW_PREDICATE_NONE);
> } else {
> diff --git a/src/intel/compiler/brw_fs_surface_builder.cpp b/src/intel/compiler/brw_fs_surface_builder.cpp
> index c346ef9e701..1d147747456 100644
> --- a/src/intel/compiler/brw_fs_surface_builder.cpp
> +++ b/src/intel/compiler/brw_fs_surface_builder.cpp
> @@ -174,7 +174,7 @@ namespace brw {
> void
> emit_byte_scattered_write(const fs_builder &bld, const fs_reg &surface,
> const fs_reg &addr, const fs_reg &src,
> - unsigned dims, unsigned size,
> + unsigned dims,
> unsigned bit_size, brw_predicate pred)
> {
> emit_send(bld, SHADER_OPCODE_BYTE_SCATTERED_WRITE_LOGICAL,
> diff --git a/src/intel/compiler/brw_fs_surface_builder.h b/src/intel/compiler/brw_fs_surface_builder.h
> index 194d61d4892..f0048220d5d 100644
> --- a/src/intel/compiler/brw_fs_surface_builder.h
> +++ b/src/intel/compiler/brw_fs_surface_builder.h
> @@ -74,7 +74,7 @@ namespace brw {
> void
> emit_byte_scattered_write(const fs_builder &bld, const fs_reg &surface,
> const fs_reg &addr, const fs_reg &src,
> - unsigned dims, unsigned size,
> + unsigned dims,
> unsigned bit_size,
> brw_predicate pred = BRW_PREDICATE_NONE);
> }
> --
> 2.14.4
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
More information about the mesa-dev
mailing list