Re: [Mesa-dev] [PATCH v2 2/2] i965/vec4_nir: Load constants as integers

2015-09-10 Thread Jason Ekstrand
On Thu, Sep 10, 2015 at 3:27 PM, Jason Ekstrand wrote: > Please add the shader-db reults to the commit message. With that, > > Reviewed-by: Jason Ekstrand > > Also, please CC stable on this one as well. I think there's some bugs > without it. In particular it fixes https://bugs.freedesktop.org

Re: [Mesa-dev] [PATCH v2 2/2] i965/vec4_nir: Load constants as integers

2015-09-10 Thread Jason Ekstrand
Please add the shader-db reults to the commit message. With that, Reviewed-by: Jason Ekstrand Also, please CC stable on this one as well. I think there's some bugs without it. --Jason On Mon, Aug 24, 2015 at 4:51 AM, Antia Puentes wrote: > Loads constants using integer as their register type

Re: [Mesa-dev] [PATCH v2 2/2] i965/vec4_nir: Load constants as integers

2015-08-25 Thread Antía Puentes
Hi! Jason, On lun, 2015-08-24 at 10:26 -0700, Jason Ekstrand wrote: > Could you please debase this patch (probably rewrite)? I *think* it > should fix https://bugs.freedesktop.org/show_bug.cgi?id=91716. What > were the shader-db results for it? This V2 of the patch is already rebased/rewritten t

Re: [Mesa-dev] [PATCH v2 2/2] i965/vec4_nir: Load constants as integers

2015-08-24 Thread Jason Ekstrand
Could you please debase this patch (probably rewrite)? I *think* it should fix https://bugs.freedesktop.org/show_bug.cgi?id=91716. What were the shader-db results for it? On Aug 24, 2015 4:51 AM, "Antia Puentes" wrote: Loads constants using integer as their register type, this is done for consis

[Mesa-dev] [PATCH v2 2/2] i965/vec4_nir: Load constants as integers

2015-08-24 Thread Antia Puentes
Loads constants using integer as their register type, this is done for consistency with the FS backend. --- src/mesa/drivers/dri/i965/brw_vec4_nir.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_vec4_nir.cpp b/src/mesa/drivers/dri/i965/brw