[Piglit] [PATCH 2/2] etc2: Add missing fragment shader float precision qualifier

Ian Romanick idr at freedesktop.org
Tue Aug 13 17:39:25 PDT 2013


On 08/13/2013 04:54 PM, Anuj Phogat wrote:
> On Fri, Aug 9, 2013 at 4:33 PM, Ian Romanick <idr at freedesktop.org> wrote:
>> From: Ian Romanick <ian.d.romanick at intel.com>
>>
>> Signed-off-by: Ian Romanick <ian.d.romanick at intel.com>
>> ---
>>   tests/spec/gles-3.0/oes_compressed_etc2_texture-miptree.c | 1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git a/tests/spec/gles-3.0/oes_compressed_etc2_texture-miptree.c b/tests/spec/gles-3.0/oes_compressed_etc2_texture-miptree.c
>> index 79a4c1f..59d8748 100644
>> --- a/tests/spec/gles-3.0/oes_compressed_etc2_texture-miptree.c
>> +++ b/tests/spec/gles-3.0/oes_compressed_etc2_texture-miptree.c
>> @@ -151,6 +151,7 @@ piglit_init(int argc, char **argv)
>>
>>          const char fs_source[] =
>>                  "#version 100\n"
>> +               "precision mediump float;\n"
>>                  "\n"
>>                  "uniform sampler2D tex;\n"
>>                  "varying vec2 tex_coord;\n"
>> --
>> 1.8.1.4
>>
>> _______________________________________________
>> Piglit mailing list
>> Piglit at lists.freedesktop.org
>> http://lists.freedesktop.org/mailman/listinfo/piglit
>
> spec/oes_compressed_etc1_rgb8_texture/oes_compressed_etc1_rgb8_texture-miptree.c
> also needs similar change. After including this change in the patch,
> this series is:
> Reviewed-by: Anuj Phogat <anuj.phogat at gmail.com>

Good catch.  I just fixed the tests that a Mesa change regressed.  That 
must mean this test (and it seems the oes_standard_derivatives tests) 
isn't running in my piglit setup.  I'll investigate that too... and send 
out fixes for the oes_standard_derivatives tests.



More information about the Piglit mailing list