glsl/lower_instructions: fix regression in dldexp_to_arith
authorIago Toral Quiroga <itoral@igalia.com>
Thu, 21 Jan 2016 09:46:39 +0000 (10:46 +0100)
committerIago Toral Quiroga <itoral@igalia.com>
Fri, 22 Jan 2016 07:14:11 +0000 (08:14 +0100)
The commit b4e198f47f842 changed the offset and bits parameters of the
bitfield insert operation from scalars to vectors. However, the lowering
of ldexp on doubles operates on each vector component and emits scalar
code (since it has to deal with the lower and upper 32-bit chunks of
each double component), so it needs its bits and offset parameters to
be scalars.

Fixes fp64 regression (crash) in:
spec/arb_gpu_shader_fp64/execution/built-in-functions/fs-ldexp-dvec4.shader_test

Reviewed-by: Ilia Mirkin <imirkin@alum.mit.edu>
src/glsl/lower_instructions.cpp

index d140be346cf041112774343a202880b2420db49b..7c2d4d7ce51bf7bafaeb286e0dc53b96c2195d22 100644 (file)
@@ -470,8 +470,8 @@ lower_instructions_visitor::dldexp_to_arith(ir_expression *ir)
 
    ir_constant *sign_mask = new(ir) ir_constant(0x80000000u);
 
-   ir_constant *exp_shift = new(ir) ir_constant(20, vec_elem);
-   ir_constant *exp_width = new(ir) ir_constant(11, vec_elem);
+   ir_constant *exp_shift = new(ir) ir_constant(20u);
+   ir_constant *exp_width = new(ir) ir_constant(11u);
    ir_constant *exp_bias = new(ir) ir_constant(1022, vec_elem);
 
    /* Temporary variables */