[Mesa-dev] [PATCH] nir: Merge redudant integer clamping.
Kenneth Graunke
kenneth at whitecape.org
Sat Apr 9 18:04:57 UTC 2016
On Saturday, April 9, 2016 3:31:16 AM PDT Markus Wick wrote:
> Dolphin uses them a lot. Range tracking would be better in the long term,
> but this two lines works fine for now.
these four lines
Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>
Thanks for fixing this! I can push the patch for you in a day or two
assuming nobody else has feedback.
> Signed-off-by: Markus Wick <markus at selfnet.de>
> ---
> src/compiler/nir/nir_opt_algebraic.py | 5 ++++-
> 1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/src/compiler/nir/nir_opt_algebraic.py b/src/compiler/nir/
nir_opt_algebraic.py
> index e72b4a7..31d7655 100644
> --- a/src/compiler/nir/nir_opt_algebraic.py
> +++ b/src/compiler/nir/nir_opt_algebraic.py
> @@ -137,7 +137,10 @@ optimizations = [
> (('~fmax', ('fmin', a, 1.0), 0.0), ('fsat', a), '!options->lower_fsat'),
> (('fsat', a), ('fmin', ('fmax', a, 0.0), 1.0), 'options->lower_fsat'),
> (('fsat', ('fsat', a)), ('fsat', a)),
> - (('fmin', ('fmax', ('fmin', ('fmax', a, 0.0), 1.0), 0.0), 1.0), ('fmin',
('fmax', a, 0.0), 1.0)),
> + (('fmin', ('fmax', ('fmin', ('fmax', a, b), c), b), c), ('fmin',
('fmax', a, b), c)),
> + (('imin', ('imax', ('imin', ('imax', a, b), c), b), c), ('imin',
('imax', a, b), c)),
> + (('umin', ('umax', ('umin', ('umax', a, b), c), b), c), ('umin',
('umax', a, b), c)),
> + (('extract_u8', ('imin', ('imax', a, 0), 0xff), 0), ('imin', ('imax', a,
0), 0xff)),
> (('~ior', ('flt', a, b), ('flt', a, c)), ('flt', a, ('fmax', b, c))),
> (('~ior', ('flt', a, c), ('flt', b, c)), ('flt', ('fmin', a, b), c)),
> (('~ior', ('fge', a, b), ('fge', a, c)), ('fge', a, ('fmin', b, c))),
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: This is a digitally signed message part.
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20160409/48cc11e2/attachment.sig>
More information about the mesa-dev
mailing list