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

Connor Abbott cwabbott0 at gmail.com
Mon May 22 04:02:47 UTC 2017


On Sun, May 21, 2017 at 11:46 PM, Connor Abbott <cwabbott0 at gmail.com> 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
>         -7.68394% +/- 1.06613%
>         (Student's t, pooled s = 15.8735)

I forgot to mention... this is for the compilation time of my
(somewhat old) copy of shader-db.

>
> Signed-off-by: Connor Abbott <cwabbott0 at gmail.com>
> ---
>  src/intel/compiler/brw_fs_validate.cpp | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/src/intel/compiler/brw_fs_validate.cpp b/src/intel/compiler/brw_fs_validate.cpp
> index 676942c..cdccf0f 100644
> --- a/src/intel/compiler/brw_fs_validate.cpp
> +++ b/src/intel/compiler/brw_fs_validate.cpp
> @@ -41,6 +41,7 @@
>  void
>  fs_visitor::validate()
>  {
> +#ifdef DEBUG
>     foreach_block_and_inst (block, fs_inst, inst, cfg) {
>        if (inst->dst.file == VGRF) {
>           fsv_assert(inst->dst.offset / REG_SIZE + regs_written(inst) <=
> @@ -54,4 +55,5 @@ fs_visitor::validate()
>           }
>        }
>     }
> +#endif
>  }
> --
> 2.9.3
>


More information about the mesa-dev mailing list