From 63250d8178dcbcf619423025e363c4f02f3bb6f9 Mon Sep 17 00:00:00 2001 From: Francisco Jerez Date: Tue, 8 Mar 2016 17:23:37 -0800 Subject: [PATCH] i965: Remove useless IR self-destruct backend_shader method. From the point it's constructed the CFG contains the only existing copy of the program IR, and it never becomes invalid. Calling backend_shader::invalidate_cfg would have destroyed the program structure irrecoverably -- We weren't calling it at all for a good reason. Reviewed-by: Kenneth Graunke --- src/mesa/drivers/dri/i965/brw_shader.cpp | 7 ------- src/mesa/drivers/dri/i965/brw_shader.h | 1 - 2 files changed, 8 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_shader.cpp b/src/mesa/drivers/dri/i965/brw_shader.cpp index dfe6afcf6d0..21977a23130 100644 --- a/src/mesa/drivers/dri/i965/brw_shader.cpp +++ b/src/mesa/drivers/dri/i965/brw_shader.cpp @@ -1046,13 +1046,6 @@ backend_shader::calculate_cfg() cfg = new(mem_ctx) cfg_t(&this->instructions); } -void -backend_shader::invalidate_cfg() -{ - ralloc_free(this->cfg); - this->cfg = NULL; -} - /** * Sets up the starting offsets for the groups of binding table entries * commong to all pipeline stages. diff --git a/src/mesa/drivers/dri/i965/brw_shader.h b/src/mesa/drivers/dri/i965/brw_shader.h index 82374a46c18..15bed78cb7c 100644 --- a/src/mesa/drivers/dri/i965/brw_shader.h +++ b/src/mesa/drivers/dri/i965/brw_shader.h @@ -217,7 +217,6 @@ public: virtual void dump_instructions(const char *name); void calculate_cfg(); - void invalidate_cfg(); virtual void invalidate_live_intervals() = 0; }; -- 2.30.2