Re: [Mesa-dev] [PATCH] i965: fix varying output setup

2016-07-22 Thread Iago Toral
This looks correct to me. Assuming no regressions in Piglit, this is: Reviewed-by: Iago Toral Quiroga On Fri, 2016-07-22 at 23:19 +1000, Timothy Arceri wrote: > Since 7f53fead5c we treat every location as using all > four components so we only need special handling for >

[Mesa-dev] [PATCH] i965: fix varying output setup

2016-07-22 Thread Timothy Arceri
Since 7f53fead5c we treat every location as using all four components so we only need special handling for doubles when they cross multiple locations. This fixes a crash in GL45-CTS.enhanced_layouts.varying_locations where the outputs array would overflow when a dmat2 was stored at the max