[PATCH v3 3/3] wined3d: Implement UAV clears on the Vulkan backend.

Jan Sikorski jsikorski at codeweavers.com
Tue Sep 28 09:43:23 CDT 2021


> On 27 Sep 2021, at 20:47, Henri Verbeet <hverbeet at gmail.com> wrote:
> 
>> @@ -5115,6 +5115,9 @@ BOOL wined3d_texture_vk_prepare_texture(struct wined3d_texture_vk *texture_vk,
>>     if (wined3d_format_is_typeless(&format_vk->f) || texture_vk->t.swapchain)
>>         flags |= VK_IMAGE_CREATE_MUTABLE_FORMAT_BIT;
>> 
>> +    if (texture_vk->t.resource.bind_flags & WINED3D_BIND_UNORDERED_ACCESS)
>> +        flags |= VK_IMAGE_CREATE_MUTABLE_FORMAT_BIT;
>> +
> This could probably be done as a separate change, and I think it
> deserves a comment that this is for creating views with a different
> format for UAV clears. We may as well use "else if", I guess.

I don’t see how it would fit in a separate change. Do you mean just setting this bit? Before the main patch it wouldn’t be used, and having the patch without it might cause crashes. Or do you mean creating the views code, in which case I guess I could put a FIXME there first?

- Jan


More information about the wine-devel mailing list