summaryrefslogtreecommitdiffstats
path: root/src/compiler/glsl/ir_clone.cpp
diff options
context:
space:
mode:
authorDanylo Piliaiev <[email protected]>2018-08-15 15:46:22 +0300
committerTimothy Arceri <[email protected]>2018-09-20 08:30:11 +1000
commit6f3c7374b11299c21d829db794fad3b756af60fb (patch)
tree5426535d63f42f086abc03152d218291a329a81d /src/compiler/glsl/ir_clone.cpp
parent194bf0a2e01769f4b29df06febf27ce340c1cd68 (diff)
glsl: Avoid propagating incompatible type of initializer
do_assignment validated assigment but when rhs type was not compatible it proceeded without issues and returned error_emitted = false. On the other hand process_initializer expected do_assignment to always return compatible type and never fail. As a result when variable was initialized with incompatible type the type of variable changed to the incompatible one. This manifested in unnecessary error messages and in one case in crash. Example GLSL: vec4 tmp = vec2(0.0); tmp.z -= 1.0; Past error messages: initializer of type vec2 cannot be assigned to variable of type vec4 invalid swizzle / mask `z' type mismatch operands to arithmetic operators must be numeric After this patch: initializer of type vec2 cannot be assigned to variable of type vec4 In the other case when we initialize variable with incompatible struct, accessing variable's field leaded to a crash. Example: uniform struct {float field;} data; ... vec4 tmp = data; tmp.x -= 1.0; After the patch there is only error line without a crash: initializer of type #anon_struct cannot be assigned to variable of type vec4 Signed-off-by: Danylo Piliaiev <[email protected]> Reviewed-by: Timothy Arceri <[email protected]> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=107547
Diffstat (limited to 'src/compiler/glsl/ir_clone.cpp')
0 files changed, 0 insertions, 0 deletions