[Mesa-dev] [PATCH 4/4] nir: add a couple of ior opts to nir_opt_algebraic

[Jason Ekstrand]

How about just adding

(('ior', 'a at bool', ('ieq', a, 0)), True)

and maybe also

(('ior', 'a at bool', ('inot', a)), True)

while you're at it.

--Jason

On Sun, Jul 15, 2018 at 4:27 PM Timothy Arceri <tarceri at itsqueeze.com>
wrote:

> One of these was seen in a Deus Ex shader.
> ---
>  src/compiler/nir/nir_opt_algebraic.py | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/src/compiler/nir/nir_opt_algebraic.py
> b/src/compiler/nir/nir_opt_algebraic.py
> index 0986dde6851..682d498b79b 100644
> --- a/src/compiler/nir/nir_opt_algebraic.py
> +++ b/src/compiler/nir/nir_opt_algebraic.py
> @@ -286,6 +286,9 @@ optimizations = [
>     (('iand', ('uge(is_used_once)', a, b), ('uge', a, c)), ('uge', a,
> ('umax', b, c))),
>     (('iand', ('uge(is_used_once)', a, c), ('uge', b, c)), ('uge',
> ('umin', a, b), c)),
>
> +   (('ior', ('flt', a, b), ('ieq', ('flt', a, b), 0)), True),
> +   (('ior', ('fge', a, b), ('ieq', ('fge', a, b), 0)), True),
> +
>     (('iand', ('ieq', 'a at 32', 0), ('ieq', 'b at 32', 0)), ('ieq', ('ior',
> 'a at 32', 'b at 32'), 0)),
>
>     # These patterns can result when (a < b || a < c) => (a < min(b, c))
> --
> 2.17.1
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20180715/e0ebb1f2/attachment.html>