[Mesa-dev] [PATCH 20/21] intel/compiler: Allocate pull_param in assign_constant_locations

[Jordan Justen]

On 2017-09-29 14:25:20, Jason Ekstrand wrote:
> diff --git a/src/intel/compiler/brw_vec4_visitor.cpp b/src/intel/compiler/brw_vec4_visitor.cpp
> index ff5cd2d..ae51619 100644
> --- a/src/intel/compiler/brw_vec4_visitor.cpp
> +++ b/src/intel/compiler/brw_vec4_visitor.cpp
> @@ -1782,6 +1782,11 @@ vec4_visitor::move_uniform_array_access_to_pull_constants()
>        return;
>     }
>  
> +   /* Allocate the pull_params array */
> +   assert(stage_prog_data->nr_pull_params == 0);
> +   stage_prog_data->pull_param = ralloc_array(mem_ctx, uint32_t,
> +                                              this->uniforms * 4);

Why allocate this based on this->uniforms if nr_pull_params is 0?

-Jordan