From 0b6da5bac66dc9abb9bd4617b156186c4fd3f92b Mon Sep 17 00:00:00 2001 From: Jason Ekstrand Date: Sat, 4 May 2019 22:47:59 -0500 Subject: [PATCH] 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 Reviewed-by: Matt Turner --- 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 1f4e122..f3c59d8 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" -- 2.7.4