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)
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
