On Fri, Sep 26, 2014 at 4:09 PM, Jason Ekstrand <ja...@jlekstrand.net> 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 <matts...@gmail.com> We should get rid of the instructions member entirely to avoid (my) mistakes like this. _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
