<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Feb 8, 2016 at 12:18 PM, 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Mon, Feb 8, 2016 at 12:01 PM, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> wrote:<br>
> On Thu, Feb 4, 2016 at 5:47 PM, Matt Turner <<a href="mailto:mattst88@gmail.com">mattst88@gmail.com</a>> wrote:<br>
>><br>
>> Prevents regressions in the next commit.<br>
>> ---<br>
>>  src/compiler/nir/nir_opt_algebraic.py | 1 +<br>
>>  1 file changed, 1 insertion(+)<br>
>><br>
>> diff --git a/src/compiler/nir/nir_opt_algebraic.py<br>
>> b/src/compiler/nir/nir_opt_algebraic.py<br>
>> index 60df69f..0a248a2 100644<br>
>> --- a/src/compiler/nir/nir_opt_algebraic.py<br>
>> +++ b/src/compiler/nir/nir_opt_algebraic.py<br>
>> @@ -167,6 +167,7 @@ optimizations = [<br>
>>     (('flog2', ('fexp2', a)), a), # lg2(2^a) = a<br>
>>     (('fpow', a, b), ('fexp2', ('fmul', ('flog2', a), b)),<br>
>> 'options->lower_fpow'), # a^b = 2^(lg2(a)*b)<br>
>>     (('fexp2', ('fmul', ('flog2', a), b)), ('fpow', a, b),<br>
>> '!options->lower_fpow'), # 2^(lg2(a)*b) = a^b<br>
>> +   (('fexp2', ('fadd', ('fmul', ('flog2', a), b), ('fmul', ('flog2', c),<br>
>> d))), ('fadd', ('fpow', a, b), ('fpow', c, d))),<br>
><br>
><br>
> I think you mean ('fmul', ('fpow', a, b), ('fpow', c, d)).  You can't pull<br>
> an add out of an exp and get another add.<br>
<br>
</span>Whoops. Yes, thank you!<br>
</blockquote></div><br></div><div class="gmail_extra">With that fixed, this one gets my R-B too.<br></div></div>