On 7 July 2011 06:20, Vadim Girlin <vadimgir...@gmail.com> wrote: > +static inline void r600_bc_src_toggle_neg(struct r600_bc_alu_src *bc_src) > +{ > + bc_src->neg = 1 - bc_src->neg; > +} > + Not necessarily wrong, but I think "bc_src->neg = !bc_src->neg;" would be the more common way to write this. _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
- [Mesa-dev] [PATCH 2/2] r600g: RSQ: clear NEG for operand Vadim Girlin
- Re: [Mesa-dev] [PATCH 2/2] r600g: RSQ: clear NEG for op... Jose Fonseca
- [Mesa-dev] [PATCH 0/2] r600g: cleanup ABS/NEG bits ... Vadim Girlin
- [Mesa-dev] [PATCH 1/2] r600g: introduce r600_bc... Vadim Girlin
- [Mesa-dev] [PATCH 2/2] r600g: introduce r600_bc... Vadim Girlin
- Re: [Mesa-dev] [PATCH 0/2] r600g: cleanup ABS/N... Vadim Girlin
- [Mesa-dev] [PATCH 0/2] [RESEND] r600g: cleanup ABS/... Vadim Girlin
- [Mesa-dev] [PATCH 1/2] r600g: introduce r600_bc... Vadim Girlin
- Re: [Mesa-dev] [PATCH 1/2] r600g: introduce... Henri Verbeet
- [Mesa-dev] [PATCH 2/2] r600g: introduce r600_bc... Vadim Girlin
- Re: [Mesa-dev] [PATCH 2/2] r600g: introduce... Henri Verbeet
- Re: [Mesa-dev] [PATCH 2/2] r600g: intr... Vadim Girlin
- [Mesa-dev] [PATCH 1/2] r600g: introduc... Vadim Girlin
- [Mesa-dev] [PATCH 2/2] r600g: introduc... Vadim Girlin
- [Mesa-dev] [PATCH 1/2] r600g: LIT: swap MUL_LIT operands to ... Vadim Girlin