On Wed, Jan 21, 2015 at 3:19 PM, Kenneth Graunke <kenn...@whitecape.org> wrote:
> Okay, I see what you're doing now.  It's definitely not what my
> brw_negate_cmod function did.  This is why you absolutely need
> doxygen comments, ideally with an example.  The functions for
> (a cmp1 b) -> !(a cmp1 b) and (a cmp1 b) -> (!a cmp2 !b) look
> way too similar at a glance...

Sure, that would have prevented confusion. I'll add

/* Returns the corresponding conditional mod for negating both operands to a
 * CMP.
 */
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to