On Fri, Dec 4, 2015 at 10:51 AM, Neil Roberts <n...@linux.intel.com> wrote:
> Matt Turner <matts...@gmail.com> writes:
>
>> # Written in the form (<search>, <replace>) where <search> is an expression
>> # and <replace> is either an expression or a value. An expression is
>> # defined as a tuple of the form (<op>, <src0>, <src1>, <src2>, <src3>)
>> @@ -94,6 +97,8 @@ optimizations = [
>> (('inot', ('ige', a, b)), ('ilt', a, b)),
>> (('inot', ('ieq', a, b)), ('ine', a, b)),
>> (('inot', ('ine', a, b)), ('ieq', a, b)),
>> + (('ieq', 'a@bool', true), a),
>> + (('ine', 'a@bool', false), a),
>
> I think this second one is already in the file on line 187 here:
>
> # Boolean simplifications
> (('ine', 'a@bool', 0), 'a'),
> (('ieq', 'a@bool', 0), ('inot', 'a')),
>
> Maybe you could add the first one near these two? It could be good to
> add a line like this as well to complete the set:
>
> (('ine', 'a@bool', true), ('inot', a)),
>
> Regards,
> - Neil
Thanks. Both are good suggestions. I've squashed them into the patch.
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
