On Dec 16, 2014 10:10 AM, "Connor Abbott" <cwabbo...@gmail.com> wrote:
>
> On Tue, Dec 16, 2014 at 1:04 AM, Jason Ekstrand <ja...@jlekstrand.net>
wrote:
> > ---
> > src/glsl/nir/nir_opcodes.h | 6 +++---
> > src/glsl/nir/nir_opt_copy_propagate.c | 2 +-
> > 2 files changed, 4 insertions(+), 4 deletions(-)
> >
> > diff --git a/src/glsl/nir/nir_opcodes.h b/src/glsl/nir/nir_opcodes.h
> > index 36a50d7..bee4bd1 100644
> > --- a/src/glsl/nir/nir_opcodes.h
> > +++ b/src/glsl/nir/nir_opcodes.h
> > @@ -186,7 +186,7 @@ BINOP(isub, nir_type_int)
> > BINOP(fmul, nir_type_float)
> > BINOP(imul, nir_type_int) /* low 32-bits of signed/unsigned integer
multiply */
> > BINOP(imul_high, nir_type_int) /* high 32-bits of signed integer
multiply */
> > -BINOP(umul_high, nir_type_int) /* high 32-bits of unsigned integer
multiply */
> > +BINOP(umul_high, nir_type_unsigned) /* high 32-bits of unsigned
integer multiply */
> >
> > BINOP(fdiv, nir_type_float)
> > BINOP(idiv, nir_type_int)
> > @@ -223,8 +223,8 @@ BINOP_COMPARE(ilt, nir_type_int)
> > BINOP_COMPARE(ige, nir_type_int)
> > BINOP_COMPARE(ieq, nir_type_int)
> > BINOP_COMPARE(ine, nir_type_int)
> > -BINOP_COMPARE(ult, nir_type_int)
> > -BINOP_COMPARE(uge, nir_type_int)
> > +BINOP_COMPARE(ult, nir_type_unsigned)
> > +BINOP_COMPARE(uge, nir_type_unsigned)
> >
> > /** integer-aware GLSL-style comparisons that compare floats and ints
*/
> > BINOP_REDUCE(ball_fequal, 1, nir_type_bool, nir_type_float)
> > diff --git a/src/glsl/nir/nir_opt_copy_propagate.c
b/src/glsl/nir/nir_opt_copy_propagate.c
> > index a2be047..b710181 100644
> > --- a/src/glsl/nir/nir_opt_copy_propagate.c
> > +++ b/src/glsl/nir/nir_opt_copy_propagate.c
> > @@ -61,7 +61,7 @@ is_swizzleless_move(nir_alu_instr *instr)
> >
> > for (unsigned i = 0; i < 4; i++) {
> > if (!((instr->dest.write_mask >> i) & 1))
> > - break;
> > + continue;
>
>
> This one isn't a typo... we know the destination is SSA here, so if
> the i'th bit of the writemask is false, then all higher bits must be
> false too. That being said, we could just replace the whole loop with
> something like:
Yeah, you pointed that out before. Since the commit message is explicitly
about typos, I'll just kill this hunk for now.
>
> for (unsigned i = 0; i < instr->dest.dest.ssa.num_components; i++) {
> if (instr->src[0].swizzle[i] != i)
> return false;
> }
>
> > if (instr->src[0].swizzle[i] != i)
> > return false;
> > }
> > --
> > 2.2.0
> >
> > _______________________________________________
> > mesa-dev mailing list
> > mesa-dev@lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/mesa-dev
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
