[Mesa-dev] Mesa (master): st/glsl_to_tgsi: simpler fixup of empty writemasks

[Nicolai Hähnle]

Hi Michel,

On 13.10.2016 08:42, Michel Dänzer wrote:
> On 13/10/16 01:50 AM, Nicolai Hähnle wrote:
>> Module: Mesa
>> Branch: master
>> Commit: f5f3cadca3809952288e3726ed5fde22090dc61d
>> URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=f5f3cadca3809952288e3726ed5fde22090dc61d
>>
>> Author: Nicolai Hähnle <nicolai.haehnle at amd.com>
>> Date:   Fri Oct  7 12:49:36 2016 +0200
>>
>> st/glsl_to_tgsi: simpler fixup of empty writemasks
>
> This change broke the piglit tests
> spec at glsl-110@execution at variable-indexing@vs-temp-array-mat2-index(-col)-wr
> on my Kaveri. Output with R600_DEBUG=ps,vs attached as
> vs-temp-array-mat2-index-wr.txt .
>
>
> P.S. The newly enabled tests
> spec at arb_enhanced_layouts@execution at component-layout@vs-tcs-load-output(-indirect)
> also fail, output attached as vs-tcs-load-output.stderr .

Thanks, it's on my radar. As far as I can tell, Mesa is doing the right 
thing here and LLVM is the problem, and working around it in Mesa may 
not be possible or reliable. But it's on my list of things to fix properly.

Nicolai