radeonsi/nir: always set input_usage_mask as using all components
authorTimothy Arceri <tarceri@itsqueeze.com>
Mon, 5 Feb 2018 22:52:47 +0000 (09:52 +1100)
committerTimothy Arceri <tarceri@itsqueeze.com>
Tue, 6 Feb 2018 21:38:52 +0000 (08:38 +1100)
This fixes a regression for now, in the future we should gather
the used components properly.

V2: just set for VS and correctly handle doubles

Fixes: be973ed21f6e "radeonsi: load the right number of components for VS inputs and TBOs"
Reviewed-by: Marek Olšák <marek.olsak@amd.com>
src/gallium/drivers/radeonsi/si_shader_nir.c

index 128be585cd17372397d0dfe14cfa80d45e2a79d7..c41caf5e5443f3603a46504bf24de14ed3919575 100644 (file)
@@ -276,14 +276,22 @@ void si_nir_scan_shader(const struct nir_shader *nir,
                unsigned attrib_count = glsl_count_attribute_slots(type,
                                                                   nir->info.stage == MESA_SHADER_VERTEX);
 
+               i = variable->data.driver_location;
+
                /* Vertex shader inputs don't have semantics. The state
                 * tracker has already mapped them to attributes via
                 * variable->data.driver_location.
                 */
                if (nir->info.stage == MESA_SHADER_VERTEX) {
-                       if (glsl_type_is_dual_slot(variable->type))
+                       /* TODO: gather the actual input useage and remove this. */
+                       info->input_usage_mask[i] = TGSI_WRITEMASK_XYZW;
+
+                       if (glsl_type_is_dual_slot(variable->type)) {
                                num_inputs += 2;
-                       else
+
+                               /* TODO: gather the actual input useage and remove this. */
+                               info->input_usage_mask[i+1] = TGSI_WRITEMASK_XYZW;
+                       } else
                                num_inputs++;
                        continue;
                }
@@ -299,8 +307,6 @@ void si_nir_scan_shader(const struct nir_shader *nir,
                        continue;
                }
 
-               i = variable->data.driver_location;
-
                for (unsigned j = 0; j < attrib_count; j++, i++) {
 
                        if (processed_inputs & ((uint64_t)1 << i))