Re: [Mesa-dev] [PATCH] i965/fs: Take into account lower frequency of conditional blocks in spilling cost heuristic.

2017-04-11 Thread Matt Turner
Nice idea. I'm happy with 0.5. It's a nice improvement already. Reviewed-by: Matt Turner ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH] i965/fs: Take into account lower frequency of conditional blocks in spilling cost heuristic.

2017-04-10 Thread Kenneth Graunke
On Monday, April 10, 2017 5:23:20 PM PDT Francisco Jerez wrote: > The individual branches of an if/else/endif construct will be executed > some unknown number of times between 0 and 1 relative to the parent > block. Use some factor in between as weight while approximating the > cost of spill/fill

Re: [Mesa-dev] [PATCH] i965/fs: Take into account lower frequency of conditional blocks in spilling cost heuristic.

2017-04-10 Thread Jason Ekstrand
On Mon, Apr 10, 2017 at 5:23 PM, Francisco Jerez wrote: > The individual branches of an if/else/endif construct will be executed > some unknown number of times between 0 and 1 relative to the parent > block. Use some factor in between as weight while approximating the >

[Mesa-dev] [PATCH] i965/fs: Take into account lower frequency of conditional blocks in spilling cost heuristic.

2017-04-10 Thread Francisco Jerez
The individual branches of an if/else/endif construct will be executed some unknown number of times between 0 and 1 relative to the parent block. Use some factor in between as weight while approximating the cost of spill/fill instructions within a conditional if-else branch. This favors spilling