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

Jason Ekstrand jason at jlekstrand.net
Mon Sep 21 13:27:13 PDT 2015


On Mon, Sep 21, 2015 at 12:40 PM, Matt Turner <mattst88 at gmail.com> wrote:
> On Mon, Sep 21, 2015 at 11:24 AM, Jason Ekstrand <jason at jlekstrand.net> 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 has ever used a "i965/compiler: " prefix. Other changes to
> brw_shader.cpp just say "i965: "
>
> The patch itself looks nice. With the prefix changed,

Done.

> Reviewed-by: Matt Turner <mattst88 at gmail.com>

Thanks!


More information about the mesa-dev mailing list