Mesa (master): intel/fs/copy-prop: Bump the hash table size to 64

[GitLab Mirror]

Module: Mesa
Branch: master
Commit: 0b6da5bac66dc9abb9bd4617b156186c4fd3f92b
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=0b6da5bac66dc9abb9bd4617b156186c4fd3f92b

Author: Jason Ekstrand <jason at jlekstrand.net>
Date:   Sat May  4 22:47:59 2019 -0500

intel/fs/copy-prop: Bump the hash table size to 64

While the number of ACPs is generally not huge compared to the number of
blocks, 16 does seem a bit small.  Bumping it to 64 takes the execution
time of the piglit vs-isnan-dvec test from about 1:18.1 on an unoptimized
debug build (what we run in CI) with NIR_VALIDATE=0 to about 1:16.2.

Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>
Reviewed-by: Matt Turner <mattst88 at gmail.com>

---

 src/intel/compiler/brw_fs_copy_propagation.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/intel/compiler/brw_fs_copy_propagation.cpp b/src/intel/compiler/brw_fs_copy_propagation.cpp
index 1f4e122e6c9..f3c59d803c2 100644
--- a/src/intel/compiler/brw_fs_copy_propagation.cpp
+++ b/src/intel/compiler/brw_fs_copy_propagation.cpp
@@ -32,7 +32,7 @@
  * 12.5 (p356).
  */
 
-#define ACP_HASH_SIZE 16
+#define ACP_HASH_SIZE 64
 
 #include "util/bitset.h"
 #include "brw_fs.h"