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

[Marek Olšák]

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.

Marek