diff options
| author | Matt Turner <mattst88@gmail.com> | 2015-03-26 10:09:54 -0700 |
|---|---|---|
| committer | Matt Turner <mattst88@gmail.com> | 2015-04-01 13:43:57 -0700 |
| commit | a03d0ba78f33e43df2ca7a9e4c58fdc9fbc9876a (patch) | |
| tree | bb08858606695ece2cacbf35286287d7af757554 /src/mesa/drivers/dri/i965/brw_fs_cmod_propagation.cpp | |
| parent | 781badee7a46c7eb778fb2755d799151d8b748bf (diff) | |
| download | external_mesa3d-a03d0ba78f33e43df2ca7a9e4c58fdc9fbc9876a.zip external_mesa3d-a03d0ba78f33e43df2ca7a9e4c58fdc9fbc9876a.tar.gz external_mesa3d-a03d0ba78f33e43df2ca7a9e4c58fdc9fbc9876a.tar.bz2 | |
i965/fs: Relax type check in cmod propagation.
The thing we want to avoid is int/float comparisons, but int/unsigned
comparisons with 0 are equivalent.
total instructions in shared programs: 6194829 -> 6193996 (-0.01%)
instructions in affected programs: 117192 -> 116359 (-0.71%)
helped: 471
Reviewed-by: Jason Ekstrand <jason.ekstrand@intel.com>
Diffstat (limited to 'src/mesa/drivers/dri/i965/brw_fs_cmod_propagation.cpp')
| -rw-r--r-- | src/mesa/drivers/dri/i965/brw_fs_cmod_propagation.cpp | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/mesa/drivers/dri/i965/brw_fs_cmod_propagation.cpp b/src/mesa/drivers/dri/i965/brw_fs_cmod_propagation.cpp index 798fef3..469f2ea 100644 --- a/src/mesa/drivers/dri/i965/brw_fs_cmod_propagation.cpp +++ b/src/mesa/drivers/dri/i965/brw_fs_cmod_propagation.cpp @@ -111,7 +111,9 @@ opt_cmod_propagation_local(bblock_t *block) break; /* Comparisons operate differently for ints and floats */ - if (scan_inst->dst.type != inst->dst.type) + if (scan_inst->dst.type != inst->dst.type && + (scan_inst->dst.type == BRW_REGISTER_TYPE_F || + inst->dst.type == BRW_REGISTER_TYPE_F)) break; /* If the instruction generating inst's source also wrote the |