Re: [Mesa-dev] [PATCH 2/2] glsl_to_tgsi: use UARL instead of I2F and ARL

2011-09-10 Thread Bryan Cain
It will only be emitted when the driver supports integer operations. The I2F+ARL combination is currently what is emitted when integer support is enabled (float targets only need ARL) but I can make that more clear in the commit message. On 09/10/2011 12:07 PM, Brian Paul wrote: > I guess my ques

Re: [Mesa-dev] [PATCH 2/2] glsl_to_tgsi: use UARL instead of I2F and ARL

2011-09-10 Thread Brian Paul
I guess my question is: do the drivers need to be updated for TGSI_OPCODE_UARL? Or will this only be emitted when the driver supports integer operations? If that's the case, please say so in the commit message or code. Otherwise: Reviewed-by: Brian Paul On 09/10/2011 10:48 AM, Bryan Cain

Re: [Mesa-dev] [PATCH 2/2] glsl_to_tgsi: use UARL instead of I2F and ARL

2011-09-10 Thread Bryan Cain
Disregard this, I meant to send this for patch 1/2 and not 2/2. This patch doesn't contain any Gallium interface changes. Bryan On 09/10/2011 11:43 AM, Bryan Cain wrote: > Can one of the Gallium interface maintainers please review this patch so > I can push it? > > Bryan > > On 09/02/2011 11:09

Re: [Mesa-dev] [PATCH 2/2] glsl_to_tgsi: use UARL instead of I2F and ARL

2011-09-10 Thread Bryan Cain
Can one of the Gallium interface maintainers please review this patch so I can push it? Bryan On 09/02/2011 11:09 AM, Bryan Cain wrote: > --- > src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 18 +++--- > 1 files changed, 7 insertions(+), 11 deletions(-) > > diff --git a/src/mesa/stat

[Mesa-dev] [PATCH 2/2] glsl_to_tgsi: use UARL instead of I2F and ARL

2011-09-02 Thread Bryan Cain
--- src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 18 +++--- 1 files changed, 7 insertions(+), 11 deletions(-) diff --git a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp b/src/mesa/state_tracker/st_glsl_to_tgsi.cpp index e2857ed..05d4d33 100644 --- a/src/mesa/state_tracker/st_glsl_to_t