[Mesa-dev] [PATCH] r600g: Merge tgsi_r600_arl and tgsi_eg_arl

Rafael Monica monraaf at gmail.com
Mon Mar 14 10:24:40 PDT 2011


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 <hverbeet at gmail.com>:
> On 14 March 2011 16:01, Rafael Monica <monraaf at 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.
>


More information about the mesa-dev mailing list