Chris Forbes <chr...@ijw.co.nz> writes: > V2: Moved up into emit(ir_texture *) to avoid duplication and fix > ordering for Gen7; Gen6 math quirks moved into previous patches. > > Tested on Gen6 only; passes all the cube_map_array piglits. > > V3: Fixed weird whitespace > > Signed-off-by: Chris Forbes <chr...@ijw.co.nz> > --- > src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 12 ++++++++++++ > 1 file changed, 12 insertions(+) > > diff --git a/src/mesa/drivers/dri/i965/brw_fs_visitor.cpp > b/src/mesa/drivers/dri/i965/brw_fs_visitor.cpp > index 571489c..b93a9e6 100644 > --- a/src/mesa/drivers/dri/i965/brw_fs_visitor.cpp > +++ b/src/mesa/drivers/dri/i965/brw_fs_visitor.cpp > @@ -1310,6 +1310,18 @@ fs_visitor::visit(ir_texture *ir) > if (ir->shadow_comparitor) > inst->shadow_compare = true; > > + /* fixup #layers for cube map arrays */ > + if (ir->op == ir_txs) { > + glsl_type const *type = ir->sampler->variable_referenced()->type;
I think you want just ir->sampler->type here, right? If you use variable_referenced, then if they have an array of cube array samplers, the yo'll ee type->base_type being GLSL_TYPE_ARRAY not GLSL_TYPE_SAMPLER
pgpBt9RkQz0eH.pgp
Description: PGP signature
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev