The patch fixes 7 piglit tests and fixes GPU lockups with 3 tests, how is that not an improvement? Perhaps FLT_TO_INT_FLOOR is more efficient than FLOOR + FLT_TO_INT, but it seems to be pretty broken. Feel free to fix it though.
2011/3/14 Henri Verbeet <hverb...@gmail.com>: > On 14 March 2011 16:01, Rafael Monica <monr...@gmail.com> wrote: >> and rename function to tgsi_arl. Fixes the following piglit tests on >> Evergreen in the process: >> >> glean >> vp1-ARL test >> shaders >> glsl-fs-varying-array >> glsl-fs-vec4-indexing-temp-src >> glsl-vs-arrays-2 >> glsl-vs-mov-after-deref >> glsl-vs-texturematrix-2 >> glsl-vs-vec4-indexing-temp-src >> >> Fixes GPU lockups on Evergreen with: >> >> shaders >> glsl-fs-vec4-indexing-temp-dst >> glsl-vs-vec4-indexing-temp-dst >> glsl-vs-varying-array >> > Why does it do that? The main thing this patch seems to do for > Evergreen is replacing FLT_TO_INT_FLOOR with FLOOR + FLT_TO_INT, but > I'm not sure that's really an improvement. > _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
