December 2022 - wine-gitlab

Re: [PATCH v2 0/8] MR1684: d3d10/effect: Further improvements to expressions support.
by Matteo Bruni (＠Mystral) 13 Dec '22

13 Dec '22

Matteo Bruni (@Mystral) commented about dlls/d3d10/effect.c: > + unsigned int i; > + > + *retval = 0.0f; > + for (i = 0; i < instr->comp_count; ++i) > + *retval += args[0][instr->scalar ? 0 : i] * args[1][i]; > +} > + > +static void pres_dotswiz(float **args, unsigned int n, const struct preshader_instr *instr) > +{ > + float *retval = args[n]; > + unsigned int i; > + > + *retval = 0.0f; > + for (i = 0; i < n; ++i) > + *retval += *args[i] * *args[i + n / 2]; > +} This doesn't look right to me either. In any case, I think we want a test for this right away. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1684#note_19578

1 0

Re: [PATCH v2 0/8] MR1684: d3d10/effect: Further improvements to expressions support.
by Matteo Bruni (＠Mystral) 13 Dec '22

13 Dec '22

Matteo Bruni (@Mystral) commented about dlls/d3d10/effect.c: > > typedef void (*pres_op_func)(float **args, unsigned int n, const struct preshader_instr *instr); > > +static void pres_mov(float **args, unsigned int n, const struct preshader_instr *instr) > +{ > + *args[1] = *args[0]; > +} > + Does this do what you want? Right now I think it just copies a pointer around. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1684#note_19577

1 0

Re: [PATCH v2 0/8] MR1684: d3d10/effect: Further improvements to expressions support.
by Matteo Bruni (＠Mystral) 13 Dec '22

13 Dec '22

Matteo Bruni (@Mystral) commented about dlls/d3d10/effect.c: > { 0x211, "bige", pres_bige }, > { 0x212, "bieq", pres_bieq }, > { 0x213, "bine", pres_bine }, > + { 0x214, "buge", pres_buge }, > + { 0x215, "bult", pres_bult }, > + { 0x219, "imul", pres_imul }, > { 0x21a, "udiv", pres_udiv }, > { 0x21e, "imax", pres_imax }, > + { 0x21f, "umin", pres_umin }, > + { 0x220, "umax", pres_umax }, Same for these. I guess you have some test in queue after this MR. Maybe it could go in early with some temporary todo_wine and error handling, if it's not too messy. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1684#note_19576

1 0

Re: [PATCH v2 0/8] MR1684: d3d10/effect: Further improvements to expressions support.
by Matteo Bruni (＠Mystral) 13 Dec '22

13 Dec '22

Matteo Bruni (@Mystral) commented about dlls/d3d10/effect.c: > { 0x204, "add", pres_add }, > { 0x205, "mul", pres_mul }, > { 0x208, "div", pres_div }, > + { 0x210, "bilt", pres_bilt }, > + { 0x211, "bige", pres_bige }, > + { 0x212, "bieq", pres_bieq }, > + { 0x213, "bine", pres_bine }, Can you add something triggering these to the tests? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1684#note_19575

1 0

Re: [PATCH v2 0/8] MR1684: d3d10/effect: Further improvements to expressions support.
by Matteo Bruni (＠Mystral) 13 Dec '22

13 Dec '22

Matteo Bruni (@Mystral) commented about dlls/d3d10/effect.c: > + unsigned int *arg1 = (unsigned int *)args[0]; > + unsigned int *arg2 = (unsigned int *)args[1]; > + float *retval = args[2]; > + unsigned int i; > + > + for (i = 0; i < instr->comp_count; ++i) > + { > + unsigned int v = arg1[instr->scalar ? 0 : i] ^ arg2[i]; > + retval[i] = *(float *)&v; > + } > +} > + > +static void pres_movc(float **args, unsigned int n, const struct preshader_instr *instr) > +{ > + float *src0 = args[0], *src1 = args[1], *src2 = args[2]; > + float *retval = args[4]; I guess this was meant to be: ```suggestion:-0+0 float *retval = args[3]; ``` -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1684#note_19574

1 0

[PATCH 0/1] MR1805: dswave: Don't force a calling convention on internal functions.
by Alex Henrie (＠alexhenrie) 13 Dec '22

13 Dec '22

I missed this one in merge request !1489. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1805

3 3

[PATCH 0/5] MR1799: rpcrt4: Avoid some unnecessary casts.
by Zebediah Figura (＠zfigura) 13 Dec '22

13 Dec '22

-- https://gitlab.winehq.org/wine/wine/-/merge_requests/1799

3 6

[PATCH 0/1] MR1788: twain_32: Handle realloc failure in twain_add_onedriver (cppcheck).
by Alex Henrie (＠alexhenrie) 13 Dec '22

13 Dec '22

-- https://gitlab.winehq.org/wine/wine/-/merge_requests/1788

3 2

[PATCH v4 0/7] MR51: vkd3d-shader/hlsl: Improve constant handling in copy prop, and extend the use of aoffimmi modifiers.
by Francisco Casas (＠fcasas) 13 Dec '22

13 Dec '22

If a hlsl_ir_load loads a variable whose components are stored from different instructions, copy propagation doesn't replace it. But if all these instructions are constants (which currently is the case for value constructors), the load can be replaced with a constant value, which is what the first patch of this series does. For instance, this shader: ``` sampler s; Texture2D t; float4 main() : sv_target { return t.Gather(s, float2(0.6, 0.6), int2(0, 0)); } ``` results in the following IR before applying the patch: ``` float | 6.00000024e-01 float | 6.00000024e-01 uint | 0 | = (<constructor-2>[@4].x @2) uint | 1 | = (<constructor-2>[@6].x @3) float2 | <constructor-2> int | 0 int | 0 uint | 0 | = (<constructor-5>[@11].x @9) uint | 1 | = (<constructor-5>[@13].x @10) int2 | <constructor-5> float4 | gather_red(resource = t, sampler = s, coords = @8, offset = @15) | return | = (<output-sv_target0> @16) ``` and this IR afterwards: ``` float2 | {6.00000024e-01 6.00000024e-01 } int2 | {0 0 } float4 | gather_red(resource = t, sampler = s, coords = @2, offset = @3) | return | = (<output-sv_target0> @4) ``` This is required to write texel_offsets as aoffimmi modifiers in the sm4 backend, since it expects the texel_offset arguments to be hlsl_ir_constant. This series also: * Allows Gather() methods to use aoffimmi modifiers instead of an additional source register (which is the only way allowed for shader model 4.1), when possible. * Adds support to texel_offsets in the Load() method via aoffimmi modifiers (the only allowed method). -- v4: vkd3d-shader/hlsl: Propagate swizzle chains in copy propagation. vkd3d-shader/hlsl: Replace swizzles with constants in copy prop. tests: Test constant propagation through swizzles. vkd3d-shader/hlsl: Support offset argument for the texture Load() method. tests: Test offset argument for the texture Load() method. vkd3d-shader/hlsl: Use aoffimmis when writing gather methods. vkd3d-shader/hlsl: Replace loads with constants in copy prop. https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/51

3 12

[PATCH 0/5] MR1750: wined3d: More Vulkan feature checks.
by Zebediah Figura (＠zfigura) 13 Dec '22

13 Dec '22

-- https://gitlab.winehq.org/wine/wine/-/merge_requests/1750

3 6

2025

2024

2023

2022

wine-gitlab December 2022