Re: [Mesa-dev] [PATCH 1/2] mesa: Don't validate IR trees on non-debug builds.

2013-03-16 Thread Kenneth Graunke
On 03/15/2013 12:04 PM, Eric Anholt wrote: This was taking 5% of CPU on TF2's load time. --- src/mesa/program/ir_to_mesa.cpp |4 1 file changed, 4 insertions(+) diff --git a/src/mesa/program/ir_to_mesa.cpp b/src/mesa/program/ir_to_mesa.cpp index 2cb5f02..ae9c0cd 100644 --- a/src/mesa

Re: [Mesa-dev] [PATCH 1/2] mesa: Don't validate IR trees on non-debug builds.

2013-03-15 Thread Jordan Justen
On Fri, Mar 15, 2013 at 12:04 PM, Eric Anholt wrote: > This was taking 5% of CPU on TF2's load time. Seems like a candidate for stable. -Jordan > --- > src/mesa/program/ir_to_mesa.cpp |4 > 1 file changed, 4 insertions(+) > > diff --git a/src/mesa/program/ir_to_mesa.cpp b/src/mesa/pro

Re: [Mesa-dev] [PATCH 1/2] mesa: Don't validate IR trees on non-debug builds.

2013-03-15 Thread Eric Anholt
Ian Romanick writes: > On 03/15/2013 12:04 PM, Eric Anholt wrote: >> This was taking 5% of CPU on TF2's load time. > > Crap... I thought we already only did this in debug mode. The series is > > Reviewed-by: Ian Romanick Turns out we'd only done it in two out of several places. pgpVd36ncfw6M

Re: [Mesa-dev] [PATCH 1/2] mesa: Don't validate IR trees on non-debug builds.

2013-03-15 Thread Ian Romanick
On 03/15/2013 12:04 PM, Eric Anholt wrote: This was taking 5% of CPU on TF2's load time. Crap... I thought we already only did this in debug mode. The series is Reviewed-by: Ian Romanick --- src/mesa/program/ir_to_mesa.cpp |4 1 file changed, 4 insertions(+) diff --git a/src/m

[Mesa-dev] [PATCH 1/2] mesa: Don't validate IR trees on non-debug builds.

2013-03-15 Thread Eric Anholt
This was taking 5% of CPU on TF2's load time. --- src/mesa/program/ir_to_mesa.cpp |4 1 file changed, 4 insertions(+) diff --git a/src/mesa/program/ir_to_mesa.cpp b/src/mesa/program/ir_to_mesa.cpp index 2cb5f02..ae9c0cd 100644 --- a/src/mesa/program/ir_to_mesa.cpp +++ b/src/mesa/program/