[Mesa-dev] [PATCH 3/6] i965/vs: When failing due to lack of spilling, don't continue on.

Eric Anholt eric at anholt.net
Wed Sep 7 13:03:34 PDT 2011


Fixes assertion failure from double-free in oglc
glsl-arrayobject constructor.declaration.structure
---
 .../drivers/dri/i965/brw_vec4_reg_allocate.cpp     |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/src/mesa/drivers/dri/i965/brw_vec4_reg_allocate.cpp b/src/mesa/drivers/dri/i965/brw_vec4_reg_allocate.cpp
index 9fd4922..1ace91f 100644
--- a/src/mesa/drivers/dri/i965/brw_vec4_reg_allocate.cpp
+++ b/src/mesa/drivers/dri/i965/brw_vec4_reg_allocate.cpp
@@ -205,6 +205,7 @@ vec4_visitor::reg_allocate()
    if (!ra_allocate_no_spills(g)) {
       ralloc_free(g);
       fail("No register spilling support yet\n");
+      return;
    }
 
    /* Get the chosen virtual registers for each node, and map virtual
-- 
1.7.5.4



More information about the mesa-dev mailing list