Re: [Mesa-dev] [PATCH] glsl/parser: handle multiple layout sections with AST nodes.

2016-05-22 Thread Chris Forbes
Eek, that would do it. Reviewed-by: Chris Forbes On Mon, May 23, 2016 at 5:55 PM, Dave Airlie wrote: > From: Dave Airlie > > For geometry/compute inputs and tess control outputs, we create > an AST node to keep track of some things. However if we have > multiple layout sections, we don't ever

Re: [Mesa-dev] [PATCH] glsl/parser: handle multiple layout sections with AST nodes.

2016-05-22 Thread Timothy Arceri
On Mon, 2016-05-23 at 15:55 +1000, Dave Airlie wrote: > From: Dave Airlie > > For geometry/compute inputs and tess control outputs, we create > an AST node to keep track of some things. However if we have > multiple layout sections, we don't ever link the node into the AST. > > This is because w

[Mesa-dev] [PATCH] glsl/parser: handle multiple layout sections with AST nodes.

2016-05-22 Thread Dave Airlie
From: Dave Airlie For geometry/compute inputs and tess control outputs, we create an AST node to keep track of some things. However if we have multiple layout sections, we don't ever link the node into the AST. This is because we create the node on the rightmost layout declaration and don't pass