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

Kenneth Graunke kenneth at whitecape.org
Sat Apr 30 07:13:31 UTC 2016


On Friday, April 29, 2016 1:29:08 PM PDT Samuel Iglesias Gonsálvez 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));
> 

Yep, math doesn't support doubles.  Let's drop this one.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: This is a digitally signed message part.
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20160430/4d2cf5c3/attachment-0001.sig>


More information about the mesa-dev mailing list