[Mesa-dev] [PATCH 11/59] i965/eu: allow doubles in math instructions

[Connor Abbott]

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
<siglesias at igalia.com> wrote:
> From: Connor Abbott <connor.w.abbott at intel.com>
>
> ---
>  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
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev