Hi Kenneth,
On Fri, 9 Sep 2011 14:41:45 -0700
Kenneth Graunke <kenn...@whitecape.org> wrote:
> - if (reg->type == BRW_REGISTER_TYPE_F) {
> - return reg->imm.f == 0.0;
> + if (type == BRW_REGISTER_TYPE_F) {
> + return imm.f == 0.0;
[...]
> - if (reg->type == BRW_REGISTER_TYPE_F) {
> - return reg->imm.f == 1.0;
> + if (type == BRW_REGISTER_TYPE_F) {
> + return imm.f == 1.0;
Shouldn't this rather be something like
fabs(imm.f - 1.0) < std::numeric_limits<float>::eps() ?
Nevertheless, I like your patch.
Best regards
Tolga Dalman
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
