On Mon, 2014-02-03 at 14:16 -0800, Tom Stellard wrote:
> 
> It's clear that there a few things that are wrong which are unrelated to this
> patch, so I think it is fine as is.  Could you add a comment above the pattern
> explaining why we need to manually toggle the sign bit and also a todo to fix 
> FabsFree
> and FNegFree.

Comments added in v3.


> If you add a lit test, then this patch has my r-b.

The v2 patch updates the two existing lit tests and adds a third one. Do
you have any other additional lit test(s) in mind?


-- 
Earthling Michel Dänzer            |                  http://www.amd.com
Libre software enthusiast          |                Mesa and X developer

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to