nir: Don't try to scalarize unpack ops.
authorEric Anholt <eric@anholt.net>
Fri, 31 Jul 2015 22:35:22 +0000 (15:35 -0700)
committerEric Anholt <eric@anholt.net>
Fri, 14 Aug 2015 18:39:18 +0000 (11:39 -0700)
Avoids regressions in vc4 when trying to do our blending in NIR.

v2: Add the other unpack ops I meant to when writing the original commit
    message.

Reviewed-by: Matt Turner <mattst88@gmail.com>
src/glsl/nir/nir_lower_alu_to_scalar.c

index 5d15fb2bc32ff8476c663566f118c2b4859d042a..efbe9e7175f71b206e36d8131e13c7088868181c 100644 (file)
@@ -100,6 +100,21 @@ lower_alu_instr_scalar(nir_alu_instr *instr, void *mem_ctx)
        */
       return;
 
+   case nir_op_unpack_unorm_4x8:
+   case nir_op_unpack_snorm_4x8:
+   case nir_op_unpack_unorm_2x16:
+   case nir_op_unpack_snorm_2x16:
+      /* There is no scalar version of these ops, unless we were to break it
+       * down to bitshifts and math (which is definitely not intended).
+       */
+      return;
+
+   case nir_op_unpack_half_2x16:
+      /* We could split this into unpack_half_2x16_split_[xy], but should
+       * we?
+       */
+      return;
+
       LOWER_REDUCTION(nir_op_fdot, nir_op_fmul, nir_op_fadd);
       LOWER_REDUCTION(nir_op_ball_fequal, nir_op_feq, nir_op_iand);
       LOWER_REDUCTION(nir_op_ball_iequal, nir_op_ieq, nir_op_iand);