diff options
| author | Jason Ekstrand <[email protected]> | 2019-05-04 22:47:59 -0500 |
|---|---|---|
| committer | Jason Ekstrand <[email protected]> | 2019-05-10 09:10:17 -0500 |
| commit | 0b6da5bac66dc9abb9bd4617b156186c4fd3f92b (patch) | |
| tree | ec7e6292edc0aa51002d33a6d1ceaeb2c56e2327 | |
| parent | ceba9ff2948d7efa57d7035c7717f046303e7c64 (diff) | |
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 <[email protected]>
Reviewed-by: Matt Turner <[email protected]>
| -rw-r--r-- | src/intel/compiler/brw_fs_copy_propagation.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
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" |