>>
>> Rune Petersen
>>
> 
> Ok commited.

I didn't look too closely at this but I've a couple of comments.
- COS looks too complicated & broken. If you'd want to get 2 with a
LOG2, you'd need 0.25 as source. But even using RCP instead, that's 5
instructions before performing the sine, for something you can easily do
in two, using another constant (just 1 add + 1 cmp needed, if you use
the right constants for the add). Maybe it's not that bad though, I
don't know how many rgb and a slots it will actually consume, but still,
are constant slots that rare?
Second, you'd really need to do range reduction of the input, otherwise
results will be very wrong for inputs outside [-pi, pi]. This would be
true for taylor approximation too, of course, unless you do an infinite
series :-). You wouldn't need to do that for SCS.

Roland


-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier.
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
--
_______________________________________________
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel

Reply via email to