Mesa (lp-binning): llvmpipe: simplify mask computation
Brian Paul
brianp at kemper.freedesktop.org
Wed Dec 2 22:16:09 UTC 2009
Module: Mesa
Branch: lp-binning
Commit: 9c486774913f66c6496cd43cfd9dbd992c28d8cb
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=9c486774913f66c6496cd43cfd9dbd992c28d8cb
Author: Brian Paul <brianp at vmware.com>
Date: Tue Dec 1 16:24:16 2009 -0700
llvmpipe: simplify mask computation
Make this a little easier to understand.
---
src/gallium/drivers/llvmpipe/lp_rast_tri.c | 10 ++++++----
1 files changed, 6 insertions(+), 4 deletions(-)
diff --git a/src/gallium/drivers/llvmpipe/lp_rast_tri.c b/src/gallium/drivers/llvmpipe/lp_rast_tri.c
index 9543b86..f6cb628 100644
--- a/src/gallium/drivers/llvmpipe/lp_rast_tri.c
+++ b/src/gallium/drivers/llvmpipe/lp_rast_tri.c
@@ -89,10 +89,12 @@ do_block_4( struct lp_rasterizer *rast,
assert(x % 4 == 0);
assert(y % 4 == 0);
- for (i = 0; i < 16; i++)
- mask |= (~(((c1 + tri->step[0][i]) |
- (c2 + tri->step[1][i]) |
- (c3 + tri->step[2][i])) >> 31)) & (1 << i);
+ for (i = 0; i < 16; i++) {
+ int any_negative = ((c1 + tri->step[0][i]) |
+ (c2 + tri->step[1][i]) |
+ (c3 + tri->step[2][i])) >> 31;
+ mask |= (~any_negative) & (1 << i);
+ }
/* As we do trivial reject already, masks should rarely be all zero:
*/
More information about the mesa-commit
mailing list