IIRC this is wrong. I think I added it in the beginning before I tested to see if the spec was lying about math instructions not being supported on doubles (it wasn't) and we can delete it now that we lower away all the possible math operations on doubles.
On Fri, Apr 29, 2016 at 7:29 AM, Samuel Iglesias Gonsálvez <[email protected]> wrote: > From: Connor Abbott <[email protected]> > > --- > src/mesa/drivers/dri/i965/brw_eu_emit.c | 6 ++++-- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/src/mesa/drivers/dri/i965/brw_eu_emit.c > b/src/mesa/drivers/dri/i965/brw_eu_emit.c > index e5057a0..4c08883 100644 > --- a/src/mesa/drivers/dri/i965/brw_eu_emit.c > +++ b/src/mesa/drivers/dri/i965/brw_eu_emit.c > @@ -1995,8 +1995,10 @@ void gen6_math(struct brw_codegen *p, > assert(src1.file == BRW_GENERAL_REGISTER_FILE || > (devinfo->gen >= 8 && src1.file == BRW_IMMEDIATE_VALUE)); > } else { > - assert(src0.type == BRW_REGISTER_TYPE_F); > - assert(src1.type == BRW_REGISTER_TYPE_F); > + assert(src0.type == BRW_REGISTER_TYPE_F || > + src0.type == BRW_REGISTER_TYPE_DF); > + assert(src1.type == BRW_REGISTER_TYPE_F || > + src1.type == BRW_REGISTER_TYPE_DF); > if (function == BRW_MATH_FUNCTION_POW) { > assert(src1.file == BRW_GENERAL_REGISTER_FILE || > (devinfo->gen >= 8 && src1.file == BRW_IMMEDIATE_VALUE)); > -- > 2.5.0 > > _______________________________________________ > mesa-dev mailing list > [email protected] > https://lists.freedesktop.org/mailman/listinfo/mesa-dev _______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
