Fixed and landed on master now.
On Mon, Sep 23, 2013 at 8:05 AM, Eric Anholt <[email protected]> wrote: > Chris Forbes <[email protected]> writes: > >> Hardware requires the magnitude of the largest component to not exceed >> 1; brw_cubemap_normalize ensures that this is the case. >> >> Unfortunately, we would previously multiply the array index for cube >> arrays by the normalization factor. The incorrect array index would then >> cause the sampler to attempt to access either the wrong cube, or memory >> outside the cube surface entirely, resulting in garbage rendering or in >> the worst case, hangs. > >> + /* coordinate.xyz *= expr */ >> + assign = new(mem_ctx) ir_assignment( >> + new(mem_ctx) ir_dereference_variable(var), >> + new(mem_ctx) ir_expression(ir_binop_mul, >> + ir->coordinate->type, >> + new(mem_ctx) ir_dereference_variable(var), >> + expr)); > > Something went weird with your indentation here. They ought to line up > with the ir_binop_mul. > > Other than that, > > Reviewed-by: Eric Anholt <[email protected]> _______________________________________________ mesa-dev mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/mesa-dev
