On 30 August 2017 at 10:38, Marek Olšák <mar...@gmail.com> wrote:
> On Wed, Aug 30, 2017 at 11:19 AM, Emil Velikov <emil.l.veli...@gmail.com> 
> wrote:
>> Hi Marek,
>>
>> There's a couple of subtle changes in the patch. If intentional, could
>> you mention that in the commit message?
>>
>> On 29 August 2017 at 23:25, Marek Olšák <mar...@gmail.com> wrote:
>>
>>>  static void r600_disk_cache_create(struct r600_common_screen *rscreen)
>>>  {
>>>         /* Don't use the cache if shader dumping is enabled. */
>>> -       if (rscreen->debug_flags &
>>> -           (DBG_FS | DBG_VS | DBG_TCS | DBG_TES | DBG_GS | DBG_PS | 
>>> DBG_CS))
>>> +       if (rscreen->debug_flags & DBG_ALL_SHADERS)
>> DBG_FS seems to be missing now.
>
> DBG_FS is a shader in r600, and r600 doesn't use the shader cache.
>
Right, thanks for the corrections.

Am I seeing it correctly, that on r600 cache files will be created
even though they're unused? Worth avoiding the creation since it's a
bit misleading/confusing?

-Emil
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to