diff options
| author | Ian Romanick <[email protected]> | 2020-05-20 13:39:57 -0700 |
|---|---|---|
| committer | Ian Romanick <[email protected]> | 2020-05-21 17:47:08 -0700 |
| commit | 685e79a64bbd6ead6f21b21ec47f55e06a8ce624 (patch) | |
| tree | 9c78ce6e7cec759310db47a0231b97e2f988bd9c /src/compiler | |
| parent | 22979f90d9587e7f31c70d07b0b8517ff0bfcaa7 (diff) | |
glsl: Remove integer matrix support from ir_dereference_array::constant_expression_value
It looks like this code has existed since day 1, but I have no idea why.
There have never been integer matrices in GLSL.
Reviewed-by: Timothy Arceri <[email protected]>
Reviewed-by: Sagar Ghuge <[email protected]>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/5135>
Diffstat (limited to 'src/compiler')
| -rw-r--r-- | src/compiler/glsl/ir_constant_expression.cpp | 10 |
1 files changed, 1 insertions, 9 deletions
diff --git a/src/compiler/glsl/ir_constant_expression.cpp b/src/compiler/glsl/ir_constant_expression.cpp index 759924ed9e1..33b097aafd4 100644 --- a/src/compiler/glsl/ir_constant_expression.cpp +++ b/src/compiler/glsl/ir_constant_expression.cpp @@ -867,13 +867,6 @@ ir_dereference_array::constant_expression_value(void *mem_ctx, ir_constant_data data = { { 0 } }; switch (column_type->base_type) { - case GLSL_TYPE_UINT: - case GLSL_TYPE_INT: - for (unsigned i = 0; i < column_type->vector_elements; i++) - data.u[i] = array->value.u[mat_idx + i]; - - break; - case GLSL_TYPE_FLOAT: for (unsigned i = 0; i < column_type->vector_elements; i++) data.f[i] = array->value.f[mat_idx + i]; @@ -887,8 +880,7 @@ ir_dereference_array::constant_expression_value(void *mem_ctx, break; default: - assert(!"Should not get here."); - break; + unreachable("Matrix types are either float or double."); } return new(mem_ctx) ir_constant(column_type, &data); |