[PATCH vkd3d 3/4] vkd3d: Don't copy an entire descriptor when destroying a view.

Conor McCarthy cmccarthy at codeweavers.com
Wed Dec 15 20:58:44 CST 2021


December 16, 2021 4:17 AM, "Henri Verbeet" <hverbeet at gmail.com> wrote:

> 
> Does this actually make a difference in the generated code or observed
> performance for you? I suppose the patch isn't wrong as such, but it
> doesn't make a difference in the generated code for me. For me, gcc
> already eliminates the dead stores in both the 32-bit and the 64-bit
> builds.

I haven't checked but see no reason for it to differ in my build. But for code quality it makes sense to copy only the view as we don't need a copy of the descriptor. The optimised copy path will do it this way.



More information about the wine-devel mailing list