On Tue, 2016-06-28 at 16:30 +0200, Iago Toral wrote: > On Tue, 2016-06-28 at 11:52 +1000, Timothy Arceri wrote: > > There are two distinctly different uses of this struct. The first > > is to store GL shader objects. The second is to store information > > about a shader stage thats been linked. > > > > The only place the new structs overlap is the shader layout fields and > > I intend to split that out into a third struct once this series lands. > > > > Having two well defined structs helps code readability and allows the > > removal > > of some unreachable code paths that were the result of confusion between > > the two uses. > > I think it is a good idea, thanks! > > I dropped a comment in patch 4, with that fixed patches 1-4 are: > Reviewed-by: Iago Toral Quiroga <ito...@igalia.com> > > I'll try to review the last 3 patches tomorrow.
Patches 5-6 are: Reviewed-by: Iago Toral Quiroga <ito...@igalia.com> Patch 4 is huge so it is kind difficult to review it in detail, it looks good to me in general but I dropped comments for few things that caught my eye. BTW, I think you should test the series against CTS too if you haven't already and maybe we should test this on something other than Intel too to be sure. Iago _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev