On Sun, May 21, 2017 at 11:46 PM, Connor Abbott <[email protected]> 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 <[email protected]> > --- > 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 > _______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
