nir: Fix nir_lower_alu_to_scalar's instr filtering.
authorEric Anholt <eric@anholt.net>
Tue, 16 Jul 2019 19:38:10 +0000 (12:38 -0700)
committerEric Anholt <eric@anholt.net>
Wed, 17 Jul 2019 17:30:43 +0000 (10:30 -0700)
It was checking if the dest or src[0] SSA values were vectors, rather than
whether the ALU op was using the source as a vector resulting in a
nir_fdot4 making it through to vc4 and v3d:

vec1 32 ssa_6 = fdot4 ssa_4.xxxx, ssa_5

Fixes: c1cffa4249ca ("nir/alu_to_scalar: Use the new NIR lowering framework")
v2: Use Jason's recommendation to look at input_sizes.

Reviewed-by: Jason Ekstrand <jason@jlekstrand.net>
src/compiler/nir/nir_lower_alu_to_scalar.c

index d346ee7a94ad272d2feab5ac32308578e6b860ab..b16624bd8aaedc5b3878b99b476b0430bb772371 100644 (file)
@@ -44,7 +44,7 @@ inst_is_vector_alu(const nir_instr *instr, const void *_state)
    assert(alu->dest.dest.is_ssa);
    assert(alu->src[0].src.is_ssa);
    return alu->dest.dest.ssa.num_components > 1 ||
-          alu->src[0].src.ssa->num_components > 1;
+          nir_op_infos[alu->op].input_sizes[0] > 1;
 }
 
 static void