Reviewed-by: Iago Toral Quiroga <ito...@igalia.com>

On Wed, 2018-02-28 at 15:11 -0800, Kenneth Graunke wrote:
> These days, we're just passing a pointer to a prog_data field, which
> we already have access to.  We can just use it directly.
> 
> (In the past, it was a pointer to a separate value.)
> ---
>  src/intel/compiler/brw_fs.cpp              | 4 ++--
>  src/intel/compiler/brw_fs.h                | 2 +-
>  src/intel/compiler/brw_fs_generator.cpp    | 4 ++--
>  src/intel/compiler/brw_shader.cpp          | 5 ++---
>  src/intel/compiler/brw_vec4.cpp            | 5 ++---
>  src/intel/compiler/brw_vec4.h              | 3 +--
>  src/intel/compiler/brw_vec4_generator.cpp  | 5 ++---
>  src/intel/compiler/brw_vec4_gs_visitor.cpp | 9 +++------
>  src/intel/compiler/brw_vec4_tcs.cpp        | 5 ++---
>  9 files changed, 17 insertions(+), 25 deletions(-)
> 
> diff --git a/src/intel/compiler/brw_fs.cpp
> b/src/intel/compiler/brw_fs.cpp
> index bed632d21b9..8f74660b59b 100644
> --- a/src/intel/compiler/brw_fs.cpp
> +++ b/src/intel/compiler/brw_fs.cpp
> @@ -6886,7 +6886,7 @@ brw_compile_fs(const struct brw_compiler
> *compiler, void *log_data,
>        prog_data->reg_blocks_0 =
> brw_register_blocks(simd16_grf_used);
>     }
>  
> -   return g.get_assembly(&prog_data->base.program_size);
> +   return g.get_assembly();
>  }
>  
>  fs_reg *
> @@ -7109,7 +7109,7 @@ brw_compile_cs(const struct brw_compiler
> *compiler, void *log_data,
>  
>        g.generate_code(cfg, prog_data->simd_size);
>  
> -      ret = g.get_assembly(&prog_data->base.program_size);
> +      ret = g.get_assembly();
>     }
>  
>     delete v8;
> diff --git a/src/intel/compiler/brw_fs.h
> b/src/intel/compiler/brw_fs.h
> index 63373580ee4..ab8d0943d15 100644
> --- a/src/intel/compiler/brw_fs.h
> +++ b/src/intel/compiler/brw_fs.h
> @@ -395,7 +395,7 @@ public:
>  
>     void enable_debug(const char *shader_name);
>     int generate_code(const cfg_t *cfg, int dispatch_width);
> -   const unsigned *get_assembly(unsigned int *assembly_size);
> +   const unsigned *get_assembly();
>  
>  private:
>     void fire_fb_write(fs_inst *inst,
> diff --git a/src/intel/compiler/brw_fs_generator.cpp
> b/src/intel/compiler/brw_fs_generator.cpp
> index cd5be054f69..2385d03ed81 100644
> --- a/src/intel/compiler/brw_fs_generator.cpp
> +++ b/src/intel/compiler/brw_fs_generator.cpp
> @@ -2261,7 +2261,7 @@ fs_generator::generate_code(const cfg_t *cfg,
> int dispatch_width)
>  }
>  
>  const unsigned *
> -fs_generator::get_assembly(unsigned int *assembly_size)
> +fs_generator::get_assembly()
>  {
> -   return brw_get_program(p, assembly_size);
> +   return brw_get_program(p, &prog_data->program_size);
>  }
> diff --git a/src/intel/compiler/brw_shader.cpp
> b/src/intel/compiler/brw_shader.cpp
> index 1df4f35cd8e..f44d82ef398 100644
> --- a/src/intel/compiler/brw_shader.cpp
> +++ b/src/intel/compiler/brw_shader.cpp
> @@ -1264,7 +1264,7 @@ brw_compile_tes(const struct brw_compiler
> *compiler,
>  
>        g.generate_code(v.cfg, 8);
>  
> -      assembly = g.get_assembly(&prog_data->base.base.program_size);
> +      assembly = g.get_assembly();
>     } else {
>        brw::vec4_tes_visitor v(compiler, log_data, key, prog_data,
>                             nir, mem_ctx, shader_time_index);
> @@ -1278,8 +1278,7 @@ brw_compile_tes(const struct brw_compiler
> *compiler,
>        v.dump_instructions();
>  
>        assembly = brw_vec4_generate_assembly(compiler, log_data,
> mem_ctx, nir,
> -                                            &prog_data->base, v.cfg,
> -                                            &prog_data-
> >base.base.program_size);
> +                                            &prog_data->base,
> v.cfg);
>     }
>  
>     return assembly;
> diff --git a/src/intel/compiler/brw_vec4.cpp
> b/src/intel/compiler/brw_vec4.cpp
> index e95886349d8..fb69ab8fe59 100644
> --- a/src/intel/compiler/brw_vec4.cpp
> +++ b/src/intel/compiler/brw_vec4.cpp
> @@ -2882,7 +2882,7 @@ brw_compile_vs(const struct brw_compiler
> *compiler, void *log_data,
>           g.enable_debug(debug_name);
>        }
>        g.generate_code(v.cfg, 8);
> -      assembly = g.get_assembly(&prog_data->base.base.program_size);
> +      assembly = g.get_assembly();
>     }
>  
>     if (!assembly) {
> @@ -2898,8 +2898,7 @@ brw_compile_vs(const struct brw_compiler
> *compiler, void *log_data,
>        }
>  
>        assembly = brw_vec4_generate_assembly(compiler, log_data,
> mem_ctx,
> -                                            shader, &prog_data-
> >base, v.cfg,
> -                                            &prog_data-
> >base.base.program_size);
> +                                            shader, &prog_data-
> >base, v.cfg);
>     }
>  
>     return assembly;
> diff --git a/src/intel/compiler/brw_vec4.h
> b/src/intel/compiler/brw_vec4.h
> index 2e93ee2946f..39ce51c7dcf 100644
> --- a/src/intel/compiler/brw_vec4.h
> +++ b/src/intel/compiler/brw_vec4.h
> @@ -45,8 +45,7 @@ brw_vec4_generate_assembly(const struct
> brw_compiler *compiler,
>                             void *mem_ctx,
>                             const nir_shader *nir,
>                             struct brw_vue_prog_data *prog_data,
> -                           const struct cfg_t *cfg,
> -                           unsigned *out_assembly_size);
> +                           const struct cfg_t *cfg);
>  
>  #ifdef __cplusplus
>  } /* extern "C" */
> diff --git a/src/intel/compiler/brw_vec4_generator.cpp
> b/src/intel/compiler/brw_vec4_generator.cpp
> index f5d6ad8e483..8852825fb22 100644
> --- a/src/intel/compiler/brw_vec4_generator.cpp
> +++ b/src/intel/compiler/brw_vec4_generator.cpp
> @@ -2221,8 +2221,7 @@ brw_vec4_generate_assembly(const struct
> brw_compiler *compiler,
>                             void *mem_ctx,
>                             const nir_shader *nir,
>                             struct brw_vue_prog_data *prog_data,
> -                           const struct cfg_t *cfg,
> -                           unsigned *out_assembly_size)
> +                           const struct cfg_t *cfg)
>  {
>     struct brw_codegen *p = rzalloc(mem_ctx, struct brw_codegen);
>     brw_init_codegen(compiler->devinfo, p, mem_ctx);
> @@ -2230,5 +2229,5 @@ brw_vec4_generate_assembly(const struct
> brw_compiler *compiler,
>  
>     generate_code(p, compiler, log_data, nir, prog_data, cfg);
>  
> -   return brw_get_program(p, out_assembly_size);
> +   return brw_get_program(p, &prog_data->base.program_size);
>  }
> diff --git a/src/intel/compiler/brw_vec4_gs_visitor.cpp
> b/src/intel/compiler/brw_vec4_gs_visitor.cpp
> index 7848832e412..fb4c1259948 100644
> --- a/src/intel/compiler/brw_vec4_gs_visitor.cpp
> +++ b/src/intel/compiler/brw_vec4_gs_visitor.cpp
> @@ -867,7 +867,7 @@ brw_compile_gs(const struct brw_compiler
> *compiler, void *log_data,
>              g.enable_debug(name);
>           }
>           g.generate_code(v.cfg, 8);
> -         return  g.get_assembly(&prog_data->base.base.program_size);
> +         return g.get_assembly();
>        }
>     }
>  
> @@ -897,9 +897,7 @@ brw_compile_gs(const struct brw_compiler
> *compiler, void *log_data,
>              /* Success! Backup is not needed */
>              ralloc_free(param);
>              return brw_vec4_generate_assembly(compiler, log_data,
> mem_ctx,
> -                                              shader, &prog_data-
> >base, v.cfg,
> -                                              &prog_data->base.base.
> -                                                  program_size);
> +                                              shader, &prog_data-
> >base, v.cfg);
>           } else {
>              /* These variables could be modified by the execution of
> the GS
>               * visitor if it packed the uniforms in the push
> constant buffer.
> @@ -962,8 +960,7 @@ brw_compile_gs(const struct brw_compiler
> *compiler, void *log_data,
>           *error_str = ralloc_strdup(mem_ctx, gs->fail_msg);
>     } else {
>        ret = brw_vec4_generate_assembly(compiler, log_data, mem_ctx,
> shader,
> -                                       &prog_data->base, gs->cfg,
> -                                       &prog_data-
> >base.base.program_size);
> +                                       &prog_data->base, gs->cfg);
>     }
>  
>     delete gs;
> diff --git a/src/intel/compiler/brw_vec4_tcs.cpp
> b/src/intel/compiler/brw_vec4_tcs.cpp
> index fdc44f1e544..80240119980 100644
> --- a/src/intel/compiler/brw_vec4_tcs.cpp
> +++ b/src/intel/compiler/brw_vec4_tcs.cpp
> @@ -487,7 +487,7 @@ brw_compile_tcs(const struct brw_compiler
> *compiler,
>  
>        g.generate_code(v.cfg, 8);
>  
> -      assembly = g.get_assembly(&prog_data->base.base.program_size);
> +      assembly = g.get_assembly();
>     } else {
>        vec4_tcs_visitor v(compiler, log_data, key, prog_data,
>                           nir, mem_ctx, shader_time_index,
> &input_vue_map);
> @@ -502,8 +502,7 @@ brw_compile_tcs(const struct brw_compiler
> *compiler,
>  
>  
>        assembly = brw_vec4_generate_assembly(compiler, log_data,
> mem_ctx, nir,
> -                                            &prog_data->base, v.cfg,
> -                                            &prog_data-
> >base.base.program_size);
> +                                            &prog_data->base,
> v.cfg);
>     }
>  
>     return assembly;
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to