Re: [Mesa-dev] [PATCH 4/6] glsl: mark xfb inputs as always_active_io

2017-10-09 Thread Timothy Arceri
On 10/10/17 10:02, Eric Anholt wrote: Timothy Arceri writes: On 10/10/17 09:31, Eric Anholt wrote: Timothy Arceri writes: We won't split varyings marked as always active because there is no point in doing so. This means we need to mark both sides of the interface as always active otherwise

Re: [Mesa-dev] [PATCH 4/6] glsl: mark xfb inputs as always_active_io

2017-10-09 Thread Eric Anholt
Timothy Arceri writes: > On 10/10/17 09:31, Eric Anholt wrote: >> Timothy Arceri writes: >> >>> We won't split varyings marked as always active because there >>> is no point in doing so. This means we need to mark both >>> sides of the interface as always active otherwise we will have >>> a mis

Re: [Mesa-dev] [PATCH 4/6] glsl: mark xfb inputs as always_active_io

2017-10-09 Thread Timothy Arceri
On 10/10/17 09:31, Eric Anholt wrote: Timothy Arceri writes: We won't split varyings marked as always active because there is no point in doing so. This means we need to mark both sides of the interface as always active otherwise we will have a mismatch and start removing things we shouldn't.

Re: [Mesa-dev] [PATCH 4/6] glsl: mark xfb inputs as always_active_io

2017-10-09 Thread Eric Anholt
Timothy Arceri writes: > We won't split varyings marked as always active because there > is no point in doing so. This means we need to mark both > sides of the interface as always active otherwise we will have > a mismatch and start removing things we shouldn't. Is this just needed because the

[Mesa-dev] [PATCH 4/6] glsl: mark xfb inputs as always_active_io

2017-10-05 Thread Timothy Arceri
We won't split varyings marked as always active because there is no point in doing so. This means we need to mark both sides of the interface as always active otherwise we will have a mismatch and start removing things we shouldn't. --- src/compiler/glsl/link_varyings.cpp | 13 ++--- 1 fil