[Mesa-dev] [PATCH] gallium/ttn: mark location specially in nir for color0-writes-all

Rob Clark robdclark at gmail.com
Sat Jun 27 14:54:22 PDT 2015


Connor, btw, how does glsl_to_nir handle this?

I guess at some point we want to try to align ttn w/ gtn (ie. cleanup
the whole nir with a tgsi accent thing)..

not that this would be the worst of the ttn vs gtn diff's in the
current state, so it is fine if there isn't a better way to do it
right now.. but I would like to start moving towards using nir
#define's for semantic names/indexes, etc, rather than tgsi or glsl
specific ones..  in the end the driver shouldn't have to care if the
nir came from tgsi/glsl/spirv/etc

BR,
-R


On Sat, Jun 27, 2015 at 5:38 PM, Ilia Mirkin <imirkin at alum.mit.edu> wrote:
> We need to distinguish a shader that has separate writes to each MRT
> from one which is supposed to write the data from MRT 0 to all the MRTs.
> In TGSI this is done with a property. NIR doesn't have that, so encode
> it as a funny location and decode on the other end.
>
> Signed-off-by: Ilia Mirkin <imirkin at alum.mit.edu>
> ---
>
> This fixes bin/fbo-drawbuffers-none gl_FragColor when I additionally
> initialize the "default" colors to a register other than r0.x -- as is
> this happens to work by luck.
>
> Also fix up vc4 to ignore this for now.
>
>  src/gallium/auxiliary/nir/tgsi_to_nir.c              | 7 ++++++-
>  src/gallium/drivers/freedreno/ir3/ir3_compiler_nir.c | 4 ++++
>  src/gallium/drivers/vc4/vc4_program.c                | 6 ++++++
>  3 files changed, 16 insertions(+), 1 deletion(-)
>
> diff --git a/src/gallium/auxiliary/nir/tgsi_to_nir.c b/src/gallium/auxiliary/nir/tgsi_to_nir.c
> index bf7eb2f..4130697 100644
> --- a/src/gallium/auxiliary/nir/tgsi_to_nir.c
> +++ b/src/gallium/auxiliary/nir/tgsi_to_nir.c
> @@ -253,7 +253,12 @@ ttn_emit_declaration(struct ttn_compile *c)
>              var->name = ralloc_asprintf(var, "out_%d", idx);
>
>              var->data.location = decl->Semantic.Name;
> -            var->data.index = decl->Semantic.Index;
> +            if (decl->Semantic.Name == TGSI_SEMANTIC_COLOR &&
> +                decl->Semantic.Index == 0 &&
> +                c->scan->properties[TGSI_PROPERTY_FS_COLOR0_WRITES_ALL_CBUFS])
> +               var->data.index = -1;
> +            else
> +               var->data.index = decl->Semantic.Index;
>
>              if (is_array) {
>                 unsigned j;
> diff --git a/src/gallium/drivers/freedreno/ir3/ir3_compiler_nir.c b/src/gallium/drivers/freedreno/ir3/ir3_compiler_nir.c
> index 3b36114..fa13c40 100644
> --- a/src/gallium/drivers/freedreno/ir3/ir3_compiler_nir.c
> +++ b/src/gallium/drivers/freedreno/ir3/ir3_compiler_nir.c
> @@ -2110,6 +2110,10 @@ setup_output(struct ir3_compile *ctx, nir_variable *out)
>                         so->writes_pos = true;
>                         break;
>                 case TGSI_SEMANTIC_COLOR:
> +                       if (semantic_index == -1) {
> +                               semantic_index = 0;
> +                               so->color0_mrt = 1;
> +                       }
>                         break;
>                 default:
>                         compile_error(ctx, "unknown FS semantic name: %s\n",
> diff --git a/src/gallium/drivers/vc4/vc4_program.c b/src/gallium/drivers/vc4/vc4_program.c
> index 2061631..728ecc6 100644
> --- a/src/gallium/drivers/vc4/vc4_program.c
> +++ b/src/gallium/drivers/vc4/vc4_program.c
> @@ -1783,6 +1783,12 @@ ntq_setup_outputs(struct vc4_compile *c)
>
>                  assert(array_len == 1);
>
> +                /* NIR hack to pass through
> +                 * TGSI_PROPERTY_FS_COLOR0_WRITES_ALL_CBUFS */
> +                if (semantic_name == TGSI_SEMANTIC_COLOR &&
> +                    semantic_index == -1)
> +                        semantic_index = 0;
> +
>                  for (int i = 0; i < 4; i++) {
>                          add_output(c,
>                                     loc + i,
> --
> 2.3.6
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev


More information about the mesa-dev mailing list