[Piglit] [PATCH] gl-1.0-blend-func: skip some blend tests when using LLVM 3.8

Roland Scheidegger rscheidegger_lists at hispeed.ch
Fri Jan 19 16:01:25 UTC 2018


Am 19.01.2018 um 06:35 schrieb Eric Anholt:
> Brian Paul <brianp at vmware.com> writes:
> 
>> On 01/18/2018 01:27 PM, Eric Anholt wrote:
>>> Brian Paul <brianp at vmware.com> writes:
>>>
>>>> To avoid an infinite loop.  See code comments for details.
>>>
>>> Skipping a failing test and returning pass is wrong to me.
>>
>> It's not ideal.  But the bug is in LLVM and cannot readily be fixed in 
>> llvmpipe.
>>
>> I could have the test return a WARN result in this situation.  Would 
>> that be better?
> 
> It's still a bug in the driver, even if it's because the driver's using
> a buggy external library.  It should be a fail.
> 

Albeit it's just a guess it will hang. With a fixed llvm 3.8 from the
stable branch, it would not hang and pass. Or IIRC if you don't have a
avx-capable cpu, it also would not hang (and with a non-x86 cpu it won't
hang neither).
But I don't really care either way if it just reports fail in this case.
(Would be nice if we could just determine the hang empirically I
suppose, if the testcase runs for more than a second kill it and it's a
fail, but that doesn't work easily.)

Roland


More information about the Piglit mailing list