Re: [Mesa-dev] [PATCHv2 10/23] i965/fs: Stop using the LOAD_PAYLOAD instruction in lower_simd_width.

2016-05-10 Thread Kenneth Graunke
On Tuesday, May 3, 2016 9:26:13 PM PDT Francisco Jerez wrote: > Instead of using the LOAD_PAYLOAD instruction (emitted through the > emit_transpose() helper that is no longer useful and this commit > removes) which had to be marked force_writemask_all in some cases, > emit a series of moves to appl

Re: [Mesa-dev] [PATCHv2 10/23] i965/fs: Stop using the LOAD_PAYLOAD instruction in lower_simd_width.

2016-05-04 Thread Francisco Jerez
Michael Schellenberger Costa writes: > Hi Curro, > > Am 04.05.2016 um 06:26 schrieb Francisco Jerez: >> Instead of using the LOAD_PAYLOAD instruction (emitted through the >> emit_transpose() helper that is no longer useful and this commit >> removes) which had to be marked force_writemask_all in

Re: [Mesa-dev] [PATCHv2 10/23] i965/fs: Stop using the LOAD_PAYLOAD instruction in lower_simd_width.

2016-05-04 Thread Michael Schellenberger Costa
Hi Curro, Am 04.05.2016 um 06:26 schrieb Francisco Jerez: > Instead of using the LOAD_PAYLOAD instruction (emitted through the > emit_transpose() helper that is no longer useful and this commit > removes) which had to be marked force_writemask_all in some cases, > emit a series of moves to apply p

Re: [Mesa-dev] [PATCHv2 10/23] i965/fs: Stop using the LOAD_PAYLOAD instruction in lower_simd_width.

2016-05-04 Thread Iago Toral
Thanks Curro! I have tested this with our fp64 branch in BDW and it works fine. FWIW, I also tested it in IVB with shader.py and did not see any regressions, so we will include it in our branch. Iago On Tue, 2016-05-03 at 21:26 -0700, Francisco Jerez wrote: > Instead of using the LOAD_PAYLOAD in

[Mesa-dev] [PATCHv2 10/23] i965/fs: Stop using the LOAD_PAYLOAD instruction in lower_simd_width.

2016-05-03 Thread Francisco Jerez
Instead of using the LOAD_PAYLOAD instruction (emitted through the emit_transpose() helper that is no longer useful and this commit removes) which had to be marked force_writemask_all in some cases, emit a series of moves to apply proper channel enable signals to the destination. Until now lower_s