nir: Generalize the optimization of subs of subs from 0.
authorEric Anholt <eric@anholt.net>
Fri, 20 Feb 2015 08:00:27 +0000 (00:00 -0800)
committerEric Anholt <eric@anholt.net>
Sat, 21 Feb 2015 22:57:14 +0000 (14:57 -0800)
I initially wrote this based on the "(('fneg', ('fneg', a)), a)" above,
but we can generalize it and make it more potentially useful.  In the
specific original case of a 0 for our new 'a' argument, it'll get further
algebraic optimization once the 0 is an argument to the new add.

No shader-db effects.

Reviewed-by: Matt Turner <mattst88@gmail.com>
Reviewed-by: Connor Abbott <cwabbott0@gmail.com>
src/glsl/nir/nir_opt_algebraic.py

index 3ea6320d36c329075456a203bdb25cd4a2af742e..7bf6431345a46198784fa7bc5e814351ed07891b 100644 (file)
@@ -151,8 +151,8 @@ optimizations = [
    (('fcsel', a, b, b), b),
 
    # Subtracts
-   (('fsub', 0.0, ('fsub', 0.0, a)), a),
-   (('isub', 0, ('isub', 0, a)), a),
+   (('fsub', a, ('fsub', 0.0, b)), ('fadd', a, b)),
+   (('isub', a, ('isub', 0, b)), ('iadd', a, b)),
    (('fneg', a), ('fsub', 0.0, a), 'options->lower_negate'),
    (('ineg', a), ('isub', 0, a), 'options->lower_negate'),
    (('fadd', a, ('fsub', 0.0, b)), ('fsub', a, b)),