<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jan 22, 2015 at 9:09 AM, Matt Turner <span dir="ltr"><<a href="mailto:mattst88@gmail.com" target="_blank">mattst88@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="">On Thu, Jan 22, 2015 at 3:41 AM, Kenneth Graunke <<a href="mailto:kenneth@whitecape.org">kenneth@whitecape.org</a>> wrote:<br>
> The GLSL IR optimization pass contained these; we may as well include<br>
> them too.<br>
><br>
> No change in the number of NIR instructions on a shader-db run.<br>
><br>
> total i965 instructions in shared programs: 6035397 -> 6035393 (-0.00%)<br>
> i965 instructions in affected programs:     772 -> 768 (-0.52%)<br>
> helped:                                     3 (all in glamor)<br>
><br>
> Signed-off-by: Kenneth Graunke <<a href="mailto:kenneth@whitecape.org">kenneth@whitecape.org</a>><br>
> ---<br>
>  src/glsl/nir/nir_opt_algebraic.py | 7 +++++++<br>
>  1 file changed, 7 insertions(+)<br>
><br>
> diff --git a/src/glsl/nir/nir_opt_algebraic.py b/src/glsl/nir/nir_opt_algebraic.py<br>
> index cf16b19..58e71e0 100644<br>
> --- a/src/glsl/nir/nir_opt_algebraic.py<br>
> +++ b/src/glsl/nir/nir_opt_algebraic.py<br>
> @@ -83,6 +83,13 @@ optimizations = [<br>
>     # DeMorgan's Laws<br>
>     (('iand', ('inot', a), ('inot', b)), ('inot', ('ior',  a, b))),<br>
>     (('ior',  ('inot', a), ('inot', b)), ('inot', ('iand', a, b))),<br>
> +   # Shift optimizations<br>
> +   (('ishl', 0, a), 0),<br>
<br>
</span>Shift zero by an unknown -> zero. Yes.<br>
<span class=""><br>
> +   (('ishl', a, 0), 0),<br>
<br>
</span>Shift an unknown by zero -> zero?!<br></blockquote><div><br></div><div>Yeah, that needs to be fixed<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
With those fixed and shader-db results confirmed,<br></blockquote><div><br></div><div>Same<br><br></div><div>Reviewed-by: Jason Ekstrand <<a href="mailto:jason.ekstrand@intel.com">jason.ekstrand@intel.com</a>></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Reviewed-by: Matt Turner <<a href="mailto:mattst88@gmail.com">mattst88@gmail.com</a>><br>
<div class=""><div class="h5">_______________________________________________<br>
mesa-dev mailing list<br>
<a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/mesa-dev" target="_blank">http://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
</div></div></blockquote></div><br></div></div>