[Mesa-dev] Allow duplicate layout qualifiers V2

Matt Turner mattst88 at gmail.com
Tue Jan 19 12:48:14 PST 2016


On Mon, Jan 18, 2016 at 12:29 AM, Timothy Arceri
<timothy.arceri at collabora.com> wrote:
> After sending out V1 where I didn't fix the global default
> merging (just issued and error), I got motivated and fixed
> it properly.
>
> There are two changes here.
>
> 1. ARB_enhanced_layouts
>
> Allows duplicates within a single layout qualifier e.g.
>
> layout(location = 0, location = 1) out vec4 a;
>
> 2. ARB_shading_language_420pack
>
> Allows multiple layout qualifiers e.g.
>
> layout(location = 0) layout(location = 2) out vec4 b;
>
>
> I've Cc'ed Matt as he gave feedback on my original change but
> I would be greatful to anyone who takes a look at these.

Patches 1 and 2 are

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

Patches 3-5 are

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

I really don't like bald true/false function arguments, but I don't
know what else to suggest.


More information about the mesa-dev mailing list