Re: [Mesa-dev] [PATCH] i965/compiler: Clean up GLSL compiler option setup

2015-09-21 Thread Jason Ekstrand
On Mon, Sep 21, 2015 at 12:40 PM, Matt Turner wrote: > On Mon, Sep 21, 2015 at 11:24 AM, Jason Ekstrand wrote: >> The only functional change here is that we now set EmitNoIndirectOutput and >> EmitNoIndirectTemp for compute shaders. Compute shaders don't have outputs >> per-se and we should have

Re: [Mesa-dev] [PATCH] i965/compiler: Clean up GLSL compiler option setup

2015-09-21 Thread Matt Turner
On Mon, Sep 21, 2015 at 11:24 AM, Jason Ekstrand wrote: > The only functional change here is that we now set EmitNoIndirectOutput and > EmitNoIndirectTemp for compute shaders. Compute shaders don't have outputs > per-se and we should have been setting EmitNoIndirectTemp all along. > --- No one h

Re: [Mesa-dev] [PATCH] i965/compiler: Clean up GLSL compiler option setup

2015-09-21 Thread Jordan Justen
Reviewed-by: Jordan Justen On 2015-09-21 11:24:59, Jason Ekstrand wrote: > The only functional change here is that we now set EmitNoIndirectOutput and > EmitNoIndirectTemp for compute shaders. Compute shaders don't have outputs > per-se and we should have been setting EmitNoIndirectTemp all alon

[Mesa-dev] [PATCH] i965/compiler: Clean up GLSL compiler option setup

2015-09-21 Thread Jason Ekstrand
The only functional change here is that we now set EmitNoIndirectOutput and EmitNoIndirectTemp for compute shaders. Compute shaders don't have outputs per-se and we should have been setting EmitNoIndirectTemp all along. --- src/mesa/drivers/dri/i965/brw_shader.cpp | 46 ++-