On Mon Feb 12 00:49:11 2024 +0000, Conor McCarthy wrote:
> The SM6 implementation is now upstream, so these todos are unnecessary
> (and were removed in an upstream commit).
You'd rather need to remove the `todo(sm<6)` annotations that Conor left behind, once (with your patches) that feature is available in the HLSL compiler.
--
https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/364#note_60986
On Sun Feb 11 16:37:35 2024 +0000, Petrichor Park wrote:
> What would make `vkd3d_string_buffer_printf` better? It looks like
> `hlsl_sprintf_alloc` just forwards to that inside and I don't see what I
> would be writing that would be better.
Because you can call `vkd3d_string_buffer_printf()` repeatedly on the same buffer, so you don't need a final `hlsl_sprintf_alloc()` just to stitch the two strings together.
--
https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/364#note_60985
This is the first part of !608, which I decided to separate to ease review.
It consists of improvements to SM1 testing, mainly:
- Correctly passing int and bool uniforms as IEEE 754 floats to SM1 profiles.
- The introduction of the "only" qualifier to avoid abusing `[require]` blocks in case tests retrieve different results for different shader models.
- Allowing some of the tests we already have for SM4/SM6 to run in SM1 after those improvements.
--
v7: tests: Remove [require] directives for tests that use int and bool uniforms.
https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/616
On Tue Feb 13 04:42:21 2024 +0000, Mohamad Al-Jaf wrote:
> I don't see how this is different from previous MRs that didn't help
> with any application, but were approved anyway like the windows.ui
> IUISettings and InputPane stub interfaces. At least in this case there's
> a possibility it would help.
Well, I'm approving because some application calls it and possibly fails because of it, but in general we prefer to only implement things that applications actually need.
Things that are called but not causing any issues can be left unimplemented, there are plenty of useless features in Windows and we don't necessarily want to implement them until they are actually needed.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/5021#note_60970