diff options
| author | Matt Turner <[email protected]> | 2017-06-30 15:48:19 -0700 |
|---|---|---|
| committer | Matt Turner <[email protected]> | 2017-07-20 16:56:50 -0700 |
| commit | aff108f2fd303ea4671541c8eca8550ddf69dc13 (patch) | |
| tree | ee453fc4f9975d87fceefa4605a887e9dfa0cdbd | |
| parent | 069bf7c9078a1b03414a6ace4bbff887e212ab90 (diff) | |
nir: Optimize find_lsb/imsb/umsb error checks
Two of the ARB_shader_ballot piglit tests hit the find_lsb case,
removing some of the noise allowed me to better debug the test when it
was failing.
Reviewed-by: Connor Abbott <[email protected]>
| -rw-r--r-- | src/compiler/nir/nir_opt_algebraic.py | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/src/compiler/nir/nir_opt_algebraic.py b/src/compiler/nir/nir_opt_algebraic.py index fe6e33d3139..df5854270c4 100644 --- a/src/compiler/nir/nir_opt_algebraic.py +++ b/src/compiler/nir/nir_opt_algebraic.py @@ -357,6 +357,17 @@ optimizations = [ (('~fadd', '#a', ('fadd', b, '#c')), ('fadd', ('fadd', a, c), b)), (('iadd', '#a', ('iadd', b, '#c')), ('iadd', ('iadd', a, c), b)), + # By definition... + (('bcsel', ('ige', ('find_lsb', a), 0), ('find_lsb', a), -1), ('find_lsb', a)), + (('bcsel', ('ige', ('ifind_msb', a), 0), ('ifind_msb', a), -1), ('ifind_msb', a)), + (('bcsel', ('ige', ('ufind_msb', a), 0), ('ufind_msb', a), -1), ('ufind_msb', a)), + + (('bcsel', ('ine', a, 0), ('find_lsb', a), -1), ('find_lsb', a)), + (('bcsel', ('ine', a, 0), ('ifind_msb', a), -1), ('ifind_msb', a)), + (('bcsel', ('ine', a, 0), ('ufind_msb', a), -1), ('ufind_msb', a)), + + (('bcsel', ('ine', a, -1), ('ifind_msb', a), -1), ('ifind_msb', a)), + # Misc. lowering (('fmod@32', a, b), ('fsub', a, ('fmul', b, ('ffloor', ('fdiv', a, b)))), 'options->lower_fmod32'), (('fmod@64', a, b), ('fsub', a, ('fmul', b, ('ffloor', ('fdiv', a, b)))), 'options->lower_fmod64'), |