diff options
| author | Timothy Arceri <[email protected]> | 2018-02-06 09:52:47 +1100 |
|---|---|---|
| committer | Timothy Arceri <[email protected]> | 2018-02-07 08:38:52 +1100 |
| commit | 1142b1d3e10815311e82cca417a62bd0b9bd21ac (patch) | |
| tree | d9c23f2e331ab182e923b4d6f4b2e3080c5b12e2 | |
| parent | ffeebcfa7eca90b62ca362a16b5f08ae44ff9ac8 (diff) | |
radeonsi/nir: always set input_usage_mask as using all components
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 <[email protected]>
| -rw-r--r-- | src/gallium/drivers/radeonsi/si_shader_nir.c | 14 |
1 files changed, 10 insertions, 4 deletions
diff --git a/src/gallium/drivers/radeonsi/si_shader_nir.c b/src/gallium/drivers/radeonsi/si_shader_nir.c index 128be585cd1..c41caf5e544 100644 --- a/src/gallium/drivers/radeonsi/si_shader_nir.c +++ b/src/gallium/drivers/radeonsi/si_shader_nir.c @@ -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)) |