On Fre, 2012-11-23 at 09:43 +0100, Christian König wrote:
> On 22.11.2012 18:57, Michel Dänzer wrote:
> > + if (si_shader_ctx->type ==
> > TGSI_PROCESSOR_VERTEX) {
> > + case TGSI_SEMANTIC_PSIZE:
> > + target = V_008DFC_SQ_EXP_POS;
> > + } else {
> > + case TGSI_SEMANTIC_STENCIL:
> > + if (d->Semantic.Name ==
> > TGSI_SEMANTIC_STENCIL)
> > + stencil_index = index;
> > + else
> > + depth_index = index;
> > + continue;
> > + }
>
> Is it just me or does this code look rather strange?
>
> Its probably just some comment or disabled part of code, otherwise I
> have my doubts that this would compile otherwise.
Of course it compiles... How could I have run piglit if it didn't? :)
The idea is to treat TGSI_SEMANTIC_POSITION the same as
TGSI_SEMANTIC_PSIZE for vertex shaders and similarly as
TGSI_SEMANTIC_STENCIL for fragment shaders.
That said, I'll concede it looks a bit weird. Feel free to follow up
with a cleanup.
--
Earthling Michel Dänzer | http://www.amd.com
Libre software enthusiast | Debian, X and DRI developer
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
