[PATCH vkd3d 08/12] vkd3d-shader/hlsl: Parse matrix constructors.

Zebediah Figura zfigura at codeweavers.com
Fri Apr 22 13:54:19 CDT 2022


On 4/22/22 05:03, Giovanni Mascellani wrote:
> Hi,
> 
> Il 21/04/22 20:57, Zebediah Figura ha scritto:
>> I dunno, I see the logic as not particularly complex...
> 
> Ok, given that Francisco reworked his patch in a rather different way, I 
> did the same for mine. I am not really sure it's the ideal solution, but 
> let's discuss it.
> 
>> I'm also slightly concerned that we're generating terrible code too 
>> zealously. Simplicity of implementation is valuable, but it means that 
>> optimization passes have more busywork to do, and this is one of those 
>> cases where I'm not sure it's actually worth it...
> 
> The optimization step required this specific case is rather simple and 
> already implemented. However, the new patch should please you better WRT 
> point of view.

Eh, I mean extra CPU work for the pass itself, but even now it's 
probably not worth worrying about.



More information about the wine-devel mailing list