On Fri, Sep 26, 2014 at 4:09 PM, Jason Ekstrand <[email protected]> wrote: > --- > src/mesa/drivers/dri/i965/brw_fs.cpp | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp > b/src/mesa/drivers/dri/i965/brw_fs.cpp > index b9bd94c..97b39e1 100644 > --- a/src/mesa/drivers/dri/i965/brw_fs.cpp > +++ b/src/mesa/drivers/dri/i965/brw_fs.cpp > @@ -3400,7 +3400,9 @@ fs_visitor::calculate_register_pressure() > invalidate_live_intervals(); > calculate_live_intervals(); > > - unsigned num_instructions = instructions.length(); > + unsigned num_instructions = 0; > + foreach_block(block, cfg) > + num_instructions = block->instructions.length();
+= > > regs_live_at_ip = rzalloc_array(mem_ctx, int, num_instructions); > > -- > 2.1.0 Oh, yeah. Nice find. Reviewed-by: Matt Turner <[email protected]> We should get rid of the instructions member entirely to avoid (my) mistakes like this. _______________________________________________ mesa-dev mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/mesa-dev
