diff options
| author | Timothy Arceri <[email protected]> | 2018-06-28 09:23:20 +1000 |
|---|---|---|
| committer | Timothy Arceri <[email protected]> | 2018-06-28 12:15:17 +1000 |
| commit | 2a5121bf355001e2c69ba05e8d9be4ed633c7bf4 (patch) | |
| tree | 03a7d52c5a0fce1bdc4cf53aa7c7699235f4f119 /src | |
| parent | e8eb182ec5980b96152ca4ce60bd5c364341b4d8 (diff) | |
glsl: skip comparison opt when adding vars of different size
The spec allows adding scalars with a vector or matrix. In this case
the opt was losing swizzle and size information.
This fixes a bug with Doom (2016) shaders.
Fixes: 34ec1a24d61f ("glsl: Optimize (x + y cmp 0) into (x cmp -y).")
Reviewed-by: Ian Romanick <[email protected]>
Diffstat (limited to 'src')
| -rw-r--r-- | src/compiler/glsl/opt_algebraic.cpp | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/src/compiler/glsl/opt_algebraic.cpp b/src/compiler/glsl/opt_algebraic.cpp index 1a8ee361652..ff4be269578 100644 --- a/src/compiler/glsl/opt_algebraic.cpp +++ b/src/compiler/glsl/opt_algebraic.cpp @@ -709,6 +709,12 @@ ir_algebraic_visitor::handle_expression(ir_expression *ir) if (!is_vec_zero(zero)) continue; + /* We are allowed to add scalars with a vector or matrix. In that + * case lets just exit early. + */ + if (add->operands[0]->type != add->operands[1]->type) + continue; + /* Depending of the zero position we want to optimize * (0 cmp x+y) into (-x cmp y) or (x+y cmp 0) into (x cmp -y) */ |