3 Sep
2008
3 Sep
'08
7:25 a.m.
2008/9/3 Jeff Zaroyko <jeffzaroyko(a)gmail.com>:
Please ignore the previous patch sent two days ago, this one fixes a performance regression as well as removing the memory leak.
-Jeff
Do you have some kind of benchmark data for this change? I don't think allocating relatively large buffers like that on the stack is a good idea. If it does make a lot of difference in terms of performance, it probably makes sense to allocate the buffer in arbfp_alloc() instead and just reuse it. In that case it probably also makes sense to do that for the ARB and GLSL shader backends as well.