[qfvis] Use unsigned for the plane side tests

Doesn't make any difference to the number of instructions, but seeing
sar instead of shr bothers me when working with bit patterns.
This commit is contained in:
Bill Currie 2021-07-29 15:25:37 +09:00
parent 6d312aaa63
commit 756214ca8e

View file

@ -247,43 +247,43 @@ split_edge (const vec4f_t *points, const vec4f_t *dists,
return mid;
}
static inline int __attribute__((const))
is_not_on (int x)
static inline unsigned __attribute__((const))
is_not_on (unsigned x)
{
return x & 1;
}
static inline int __attribute__((const))
is_back (int x)
static inline unsigned __attribute__((const))
is_back (unsigned x)
{
return x & 2;
}
static inline int __attribute__((const))
static inline unsigned __attribute__((const))
is_not_back (unsigned x)
{
return ~x & 2;
}
static inline int __attribute__((const))
static inline unsigned __attribute__((const))
is_front (unsigned x)
{
return is_not_on (x) & (is_not_back (x) >> 1);
}
static inline int __attribute__((const))
static inline unsigned __attribute__((const))
is_back_front (unsigned x, unsigned y)
{
return (is_back (x) >> 1) & is_front (y);
}
static inline int __attribute__((const))
static inline unsigned __attribute__((const))
is_front_back (unsigned x, unsigned y)
{
return is_front (x) & (is_back (y) >> 1);
}
static inline int __attribute__((const))
static inline unsigned __attribute__((const))
is_transition (unsigned x, unsigned y)
{
return is_back_front (x, y) | is_front_back (x, y);