[Mesa-dev] [PATCH 1/2] intel/fs: Alloc pull constants off mem_ctx

[Lionel Landwerlin]

Both patches are :

Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin at intel.com>

On 01/11/17 16:00, Jason Ekstrand wrote:
> It doesn't actually matter since the only user of push constants, i965,
> ralloc_steals it back to NULL but it's more consistent and probably
> fixes memory leaks in some error cases.
> ---
>   src/intel/compiler/brw_fs.cpp | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/intel/compiler/brw_fs.cpp b/src/intel/compiler/brw_fs.cpp
> index e546792..21ff030 100644
> --- a/src/intel/compiler/brw_fs.cpp
> +++ b/src/intel/compiler/brw_fs.cpp
> @@ -2095,7 +2095,7 @@ fs_visitor::assign_constant_locations()
>      stage_prog_data->param = ralloc_array(mem_ctx, uint32_t, num_push_constants);
>      if (num_pull_constants > 0) {
>         stage_prog_data->nr_pull_params = num_pull_constants;
> -      stage_prog_data->pull_param = ralloc_array(NULL, uint32_t,
> +      stage_prog_data->pull_param = ralloc_array(mem_ctx, uint32_t,
>                                                    num_pull_constants);
>      }
>