Chris Forbes <chr...@ijw.co.nz> writes:

> Signed-off-by: Chris Forbes <chr...@ijw.co.nz>
> ---
>  src/mesa/drivers/dri/i915/i915_state.c | 9 +++++----
>  src/mesa/drivers/dri/i965/brw_cc.c     | 5 +++--
>  src/mesa/drivers/dri/i965/gen6_cc.c    | 5 +++--
>  3 files changed, 11 insertions(+), 8 deletions(-)
>
> diff --git a/src/mesa/drivers/dri/i915/i915_state.c 
> b/src/mesa/drivers/dri/i915/i915_state.c
> index 98eac8d..2fd0bf1 100644
> --- a/src/mesa/drivers/dri/i915/i915_state.c
> +++ b/src/mesa/drivers/dri/i915/i915_state.c

> diff --git a/src/mesa/drivers/dri/i965/brw_cc.c 
> b/src/mesa/drivers/dri/i965/brw_cc.c
> index 988b85e..71f9366 100644
> --- a/src/mesa/drivers/dri/i965/brw_cc.c
> +++ b/src/mesa/drivers/dri/i965/brw_cc.c
> @@ -35,6 +35,7 @@
>  #include "brw_defines.h"
>  #include "brw_util.h"
>  #include "main/macros.h"
> +#include "main/stencil.h"
>  #include "intel_batchbuffer.h"
>  
>  static void
> @@ -116,7 +117,7 @@ static void upload_cc_unit(struct brw_context *brw)
>        intel_translate_stencil_op(ctx->Stencil.ZFailFunc[0]);
>        cc->cc0.stencil_pass_depth_pass_op =
>        intel_translate_stencil_op(ctx->Stencil.ZPassFunc[0]);
> -      cc->cc1.stencil_ref = ctx->Stencil.Ref[0];
> +      cc->cc1.stencil_ref = _mesa_get_stencil_ref(ctx, 0);

I'm not sure all these driver changes are necessary -- they're under
Stencil._Enabled, which is checking whether you have 0 or 8 bits of
stencil, and otherwise they'll all be truncated to their lower 8 bits
because of the size of these fields, right?

Attachment: pgp5wJ51fsN6p.pgp
Description: PGP signature

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to