diff options
| author | Jason Ekstrand <[email protected]> | 2017-11-01 07:57:21 -0700 |
|---|---|---|
| committer | Jason Ekstrand <[email protected]> | 2017-11-02 09:55:21 -0700 |
| commit | 7b4387519c382cffef9c62bbbbefcfe71cfde905 (patch) | |
| tree | 80efd8f4a46f2c1a28b104627ca4789523fbeafa /src/intel/compiler | |
| parent | cb831d918b8151621ae405bc63c011eae0339e60 (diff) | |
intel/fs: Alloc pull constants off mem_ctx
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.
Reviewed-by: Lionel Landwerlin <[email protected]>
Reviewed-by: Tapani Pälli <[email protected]>
Cc: [email protected]
Diffstat (limited to 'src/intel/compiler')
| -rw-r--r-- | src/intel/compiler/brw_fs.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/intel/compiler/brw_fs.cpp b/src/intel/compiler/brw_fs.cpp index 814da8ad94d..e2176827a64 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); } |