Re: [Mesa-dev] [PATCH] i965/fs: don't do IR validation in release mode

2017-05-23 Thread Thomas Helland
This looks to be in line with what I've seen with my profiling. I was wondering why we where doing validation when debug was not enabled. Reviewed-by: Thomas Helland 2017-05-22 5:46 GMT+02:00 Connor Abbott : > Surprisingly, profiling shows that this takes quite a lot of time. So > turn it off wh

Re: [Mesa-dev] [PATCH] i965/fs: don't do IR validation in release mode

2017-05-21 Thread Connor Abbott
On Sun, May 21, 2017 at 11:46 PM, Connor Abbott wrote: > Surprisingly, profiling shows that this takes quite a lot of time. So > turn it off when building in release mode, just like all the other > validation passes we do. > > Difference at 95.0% confidence > -82.5956 +/- 11.46 > -

[Mesa-dev] [PATCH] i965/fs: don't do IR validation in release mode

2017-05-21 Thread Connor Abbott
Surprisingly, profiling shows that this takes quite a lot of time. So turn it off when building in release mode, just like all the other validation passes we do. Difference at 95.0% confidence -82.5956 +/- 11.46 -7.68394% +/- 1.06613% (Student's t, pooled s = 15.8735) Sign