Re: [Mesa-dev] [PATCH 20/22] i965/vec4: Relax writemask condition in CSE

2018-03-07 Thread Kenneth Graunke
On Tuesday, March 6, 2018 11:10:57 AM PST Ian Romanick wrote: > On 03/05/2018 02:50 PM, Kenneth Graunke wrote: > > On Friday, February 23, 2018 3:56:05 PM PST Ian Romanick wrote: > >> From: Ian Romanick > >> > >> If the previously seen instruction generates more fields

Re: [Mesa-dev] [PATCH 20/22] i965/vec4: Relax writemask condition in CSE

2018-03-06 Thread Ian Romanick
On 03/05/2018 02:50 PM, Kenneth Graunke wrote: > On Friday, February 23, 2018 3:56:05 PM PST Ian Romanick wrote: >> From: Ian Romanick >> >> If the previously seen instruction generates more fields than the new >> instruction, still allow CSE to happen. This doesn't do

Re: [Mesa-dev] [PATCH 20/22] i965/vec4: Relax writemask condition in CSE

2018-03-05 Thread Kenneth Graunke
On Friday, February 23, 2018 3:56:05 PM PST Ian Romanick wrote: > From: Ian Romanick > > If the previously seen instruction generates more fields than the new > instruction, still allow CSE to happen. This doesn't do much, but it > also enables a couple more shaders in

[Mesa-dev] [PATCH 20/22] i965/vec4: Relax writemask condition in CSE

2018-02-23 Thread Ian Romanick
From: Ian Romanick If the previously seen instruction generates more fields than the new instruction, still allow CSE to happen. This doesn't do much, but it also enables a couple more shaders in the next patch. It helped quite a bit in another change series that I