Re: [Mesa-dev] [PATCH] radv: call constant folding before opt algebraic

2019-05-08 Thread Samuel Pitoiset
LGTM. Thanks for double checking Tim! On 5/8/19 1:27 AM, Timothy Arceri wrote: On 8/5/19 1:51 am, Samuel Pitoiset wrote: What games are affected btw?  PERCENTAGE DELTAS    Shaders SGPRs VGPRs SpillSGPR CodeSize MaxWaves  batman-arkham-city   2581 . . . .

Re: [Mesa-dev] [PATCH] radv: call constant folding before opt algebraic

2019-05-07 Thread Timothy Arceri
On 8/5/19 1:51 am, Samuel Pitoiset wrote: What games are affected btw? PERCENTAGE DELTASShaders SGPRs VGPRs SpillSGPR CodeSize MaxWaves batman-arkham-city 2581 . . .. . dawn-of-war-3 244 . . ..

Re: [Mesa-dev] [PATCH] radv: call constant folding before opt algebraic

2019-05-07 Thread Samuel Pitoiset
What games are affected btw? Can you please double check before pushing because of the flrp changes that landed around? On 5/7/19 7:14 AM, Timothy Arceri wrote: ping! On 2/5/19 1:38 pm, Timothy Arceri wrote: The pattern of calling opt algebraic first seems to have originated in i965. The

Re: [Mesa-dev] [PATCH] radv: call constant folding before opt algebraic

2019-05-07 Thread Bas Nieuwenhuizen
Nope, r-b On Tue, May 7, 2019 at 8:36 AM Samuel Pitoiset wrote: > > Seems fine to, > > Reviewed-by: Samuel Pitoiset > > Bas, any comments? > > On 5/7/19 7:14 AM, Timothy Arceri wrote: > > ping! > > > > On 2/5/19 1:38 pm, Timothy Arceri wrote: > >> The pattern of calling opt algebraic first

Re: [Mesa-dev] [PATCH] radv: call constant folding before opt algebraic

2019-05-07 Thread Samuel Pitoiset
Seems fine to, Reviewed-by: Samuel Pitoiset Bas, any comments? On 5/7/19 7:14 AM, Timothy Arceri wrote: ping! On 2/5/19 1:38 pm, Timothy Arceri wrote: The pattern of calling opt algebraic first seems to have originated in i965. The order in OpenGL drivers generally doesn't matter because

Re: [Mesa-dev] [PATCH] radv: call constant folding before opt algebraic

2019-05-06 Thread Timothy Arceri
ping! On 2/5/19 1:38 pm, Timothy Arceri wrote: The pattern of calling opt algebraic first seems to have originated in i965. The order in OpenGL drivers generally doesn't matter because the GLSL IR optimisations do constant folding before opt algebraic. However in Vulkan drivers calling opt

[Mesa-dev] [PATCH] radv: call constant folding before opt algebraic

2019-05-01 Thread Timothy Arceri
The pattern of calling opt algebraic first seems to have originated in i965. The order in OpenGL drivers generally doesn't matter because the GLSL IR optimisations do constant folding before opt algebraic. However in Vulkan drivers calling opt algebraic first can result in missed constant folding