diff options
| author | Matt Turner <mattst88@gmail.com> | 2016-04-20 14:22:53 -0700 |
|---|---|---|
| committer | Matt Turner <mattst88@gmail.com> | 2016-04-21 10:32:40 -0700 |
| commit | 33565d67641142a68d537023e181b6dcd587e551 (patch) | |
| tree | 45511efd04452dc6e996890c165259eb8fe60290 /src/mesa/drivers/dri/i965/brw_fs.h | |
| parent | 0020ca3c9269a8baa54501cb824af14fa497ac14 (diff) | |
| download | external_mesa3d-33565d67641142a68d537023e181b6dcd587e551.zip external_mesa3d-33565d67641142a68d537023e181b6dcd587e551.tar.gz external_mesa3d-33565d67641142a68d537023e181b6dcd587e551.tar.bz2 | |
i965/fs: Readd opt_drop_redundant_mov_to_flags().
This reverts commit b449366587b5f3f64c6fb45fe22c39e4bc8a4309.
I removed the pass thinking that it was now not useful, but that was not
true. I believe I ran shader-db on HSW and saw no results, but HSW does
not use the unlit centroid workaround code and as a result does not emit
redundant MOV_DISPATCH_TO_FLAGS instructions.
On IVB, the shader-db results are:
total instructions in shared programs: 6650806 -> 6646303 (-0.07%)
instructions in affected programs: 106893 -> 102390 (-4.21%)
helped: 793
total cycles in shared programs: 56195538 -> 56103720 (-0.16%)
cycles in affected programs: 873048 -> 781230 (-10.52%)
helped: 553
HURT: 209
On SNB, the shader-db results are:
total instructions in shared programs: 7173074 -> 7168541 (-0.06%)
instructions in affected programs: 119757 -> 115224 (-3.79%)
helped: 799
total cycles in shared programs: 98128032 -> 98072938 (-0.06%)
cycles in affected programs: 1437104 -> 1382010 (-3.83%)
helped: 454
HURT: 237
Reviewed-by: Iago Toral Quiroga <itoral@igalia.com>
Diffstat (limited to 'src/mesa/drivers/dri/i965/brw_fs.h')
| -rw-r--r-- | src/mesa/drivers/dri/i965/brw_fs.h | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/mesa/drivers/dri/i965/brw_fs.h b/src/mesa/drivers/dri/i965/brw_fs.h index c7f20ca..bcd2e3e 100644 --- a/src/mesa/drivers/dri/i965/brw_fs.h +++ b/src/mesa/drivers/dri/i965/brw_fs.h @@ -153,6 +153,7 @@ public: bool try_constant_propagate(fs_inst *inst, acp_entry *entry); bool opt_copy_propagate_local(void *mem_ctx, bblock_t *block, exec_list *acp); + bool opt_drop_redundant_mov_to_flags(); bool opt_register_renaming(); bool register_coalesce(); bool compute_to_mrf(); |