[Mesa-dev] [PATCH] radv/ac: workaround regression in llvm 4.0 release

[Emil Velikov]

On 30 August 2017 at 17:59, Marek Olšák <maraeo at gmail.com> wrote:
> On Wed, Aug 30, 2017 at 6:36 PM, Emil Velikov <emil.l.velikov at gmail.com> wrote:
>> Hi all,
>>
>> On 14 March 2017 at 23:51, Samuel Pitoiset <samuel.pitoiset at gmail.com> wrote:
>>> On 03/14/2017 10:21 PM, Dave Airlie wrote:
>>>>
>>>> On 15 March 2017 at 07:17, Dave Airlie <airlied at gmail.com> wrote:
>>>>>
>>>>> From: Dave Airlie <airlied at redhat.com>
>>>>>
>>>>> LLVM 4.0 released with a pretty messy regression, that hopefully
>>>>> get fixed in the future.
>>>>>
>>>>> This work around was proposed by Tom, and it fixes the CTS regressions
>>>>> here at least, I'm not sure if this will cause any major side effects,
>>>>> but correctness over speed and all that.
>>>>>
>>>>> radeonsi should possibly consider the same workaround until an llvm
>>>>> fix can be found.
>>>>>
>>>>
>>>> I also deliberately only put this for LLVM 4.0 in the hope llvm will
>>>> get fixed before
>>>> 5 releases.
>>>
>>>
>>> I think you want to make it >= 0x0400 and change it later because it's still
>>> broken with LLVM master.
>>>
>> Did this fix make it for LLVM 5.0 or LLVM 4.0.1?
>> If so, let's adjust the guard accordingly.
>
> I think there are more issues the LLVM option fixes, so I think we
> need to keep that.
>
Thanks for the confirmation Marek. Hopefully it will get sorted one of
these days.

-Emil