diff options
| author | Samuel Pitoiset <samuel.pitoiset@gmail.com> | 2018-07-20 18:48:07 +0200 |
|---|---|---|
| committer | Samuel Pitoiset <samuel.pitoiset@gmail.com> | 2018-07-23 11:34:39 +0200 |
| commit | 6e32d9e7b0945268b844ab2397ab3c9924136d3a (patch) | |
| tree | a12dacff4045981b3c6397159e4ec97473443231 | |
| parent | d0ee0a0a5d5fb658c87fed82cbcf9b48437abb2c (diff) | |
| download | external_mesa3d-6e32d9e7b0945268b844ab2397ab3c9924136d3a.tar.gz external_mesa3d-6e32d9e7b0945268b844ab2397ab3c9924136d3a.tar.bz2 external_mesa3d-6e32d9e7b0945268b844ab2397ab3c9924136d3a.zip | |
radv: fix a memleak for merged shaders on GFX9
modules[i] can be NULL for merged shaders but we have to
free the NIR code. radv_can_dump_shader_stats() already handles
if modules[i] is NULL, no need to check it twice.
Cc: mesa-stable@lists.freedesktop.org
Signed-off-by: Samuel Pitoiset <samuel.pitoiset@gmail.com>
Reviewed-by: Bas Nieuwenhuizen <bas@basnieuwenhuizen.nl>
| -rw-r--r-- | src/amd/vulkan/radv_pipeline.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/amd/vulkan/radv_pipeline.c b/src/amd/vulkan/radv_pipeline.c index 27e13a2251c..545b618b7c2 100644 --- a/src/amd/vulkan/radv_pipeline.c +++ b/src/amd/vulkan/radv_pipeline.c @@ -2160,7 +2160,7 @@ void radv_create_shaders(struct radv_pipeline *pipeline, for (int i = 0; i < MESA_SHADER_STAGES; ++i) { free(codes[i]); - if (modules[i]) { + if (nir[i]) { if (!pipeline->device->keep_shader_info) ralloc_free(nir[i]); |