Skip to content
This repository has been archived by the owner on Jan 7, 2023. It is now read-only.

Commit

Permalink
intel/fs/copy-prop: Bump the hash table size to 64
Browse files Browse the repository at this point in the history
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@whitecape.org>
Reviewed-by: Matt Turner <mattst88@gmail.com>
  • Loading branch information
gfxstrand committed May 10, 2019
1 parent ceba9ff commit 0b6da5b
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/intel/compiler/brw_fs_copy_propagation.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -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"
Expand Down

0 comments on commit 0b6da5b

Please sign in to comment.