September 2020 - Wine-Commits

Piotr Caban : msasn1: Fix buffer allocations.
by Alexandre Julliard Sept. 18, 2020

Sept. 18, 2020

Module: wine Branch: master Commit: d101fdb4499e174498c6378d1e47950113795009 URL: https://source.winehq.org/git/wine.git/?a=commit;h=d101fdb4499e174498c6378d… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Fri Sep 18 14:30:23 2020 +0200 msasn1: Fix buffer allocations. Signed-off-by: Piotr Caban <piotr(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/msasn1/main.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/dlls/msasn1/main.c b/dlls/msasn1/main.c index d151362905..fc867e7b81 100644 --- a/dlls/msasn1/main.c +++ b/dlls/msasn1/main.c @@ -56,7 +56,7 @@ ASN1module_t WINAPI ASN1_CreateModule(ASN1uint32_t ver, ASN1encodingrule_e rule, if (!encoder || !decoder || !freemem || !size) return module; - module = heap_alloc(sizeof(module)); + module = heap_alloc(sizeof(*module)); if (module) { module->nModuleName = magic; @@ -103,7 +103,7 @@ ASN1error_e WINAPI ASN1_CreateEncoder(ASN1module_t module, ASN1encoding_t *encod if (!module || !encoder) return ASN1_ERR_BADARGS; - enc = heap_alloc(sizeof(enc)); + enc = heap_alloc(sizeof(*enc)); if (!enc) { return ASN1_ERR_MEMORY; @@ -153,7 +153,7 @@ ASN1error_e WINAPI ASN1_CreateDecoder(ASN1module_t module, ASN1decoding_t *decod if (!module || !decoder) return ASN1_ERR_BADARGS; - dec = heap_alloc(sizeof(dec)); + dec = heap_alloc(sizeof(*dec)); if (!dec) { return ASN1_ERR_MEMORY;

1 0

Kevin Puetz : oleaut32/tests: Fix expect_wstr_acpval(...,NULL).
by Alexandre Julliard Sept. 18, 2020

Sept. 18, 2020

Module: wine Branch: master Commit: b43041d79af94e92c78f1897a6e4c95a4686b8f9 URL: https://source.winehq.org/git/wine.git/?a=commit;h=b43041d79af94e92c78f1897… Author: Kevin Puetz <PuetzKevinA(a)JohnDeere.com> Date: Thu Sep 17 18:24:17 2020 -0500 oleaut32/tests: Fix expect_wstr_acpval(...,NULL). A NULL expected value is always a test failure, but printing the unexpected "wrong" value is more helpful than crashing. Signed-off-by: Kevin Puetz <PuetzKevinA(a)JohnDeere.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/oleaut32/tests/typelib.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/oleaut32/tests/typelib.c b/dlls/oleaut32/tests/typelib.c index 3d5aeb0a26..6f68a201ba 100644 --- a/dlls/oleaut32/tests/typelib.c +++ b/dlls/oleaut32/tests/typelib.c @@ -53,7 +53,7 @@ { \ CHAR buf[260]; \ expect_eq(!WideCharToMultiByte(CP_ACP, 0, (expr), -1, buf, 260, NULL, NULL), 0, int, "%d"); \ - ok(strcmp(value, buf) == 0, #expr " expected \"%s\" got \"%s\"\n", value, buf); \ + ok(value && strcmp(value, buf) == 0, #expr " expected \"%s\" got \"%s\"\n", value, buf); \ } #define ole_expect(expr, expect) { \

1 0

Alexandre Julliard : include: Don't use IsBadStringPtr in debug functions for Unix libraries.
by Alexandre Julliard Sept. 18, 2020

Sept. 18, 2020

Module: wine Branch: master Commit: 2eed0cdb8f860d37f4efa631719a0a9ae4f721e1 URL: https://source.winehq.org/git/wine.git/?a=commit;h=2eed0cdb8f860d37f4efa631… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Sep 18 15:13:19 2020 +0200 include: Don't use IsBadStringPtr in debug functions for Unix libraries. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ntdll/unix/virtual.c | 44 -------------------------------------------- include/wine/debug.h | 4 ++++ 2 files changed, 4 insertions(+), 44 deletions(-) diff --git a/dlls/ntdll/unix/virtual.c b/dlls/ntdll/unix/virtual.c index 22d328b1ce..62b2ca1346 100644 --- a/dlls/ntdll/unix/virtual.c +++ b/dlls/ntdll/unix/virtual.c @@ -3201,50 +3201,6 @@ BOOL virtual_check_buffer_for_write( void *ptr, SIZE_T size ) } -/************************************************************* - * IsBadStringPtrA - * - * IsBadStringPtrA replacement for ntdll, to catch exception in debug traces. - */ -BOOL WINAPI IsBadStringPtrA( LPCSTR str, UINT_PTR max ) -{ - if (!str) return TRUE; - __TRY - { - volatile const char *p = str; - while (p != str + max) if (!*p++) break; - } - __EXCEPT_SYSCALL - { - return TRUE; - } - __ENDTRY - return FALSE; -} - - -/************************************************************* - * IsBadStringPtrW - * - * IsBadStringPtrW replacement for ntdll, to catch exception in debug traces. - */ -BOOL WINAPI IsBadStringPtrW( LPCWSTR str, UINT_PTR max ) -{ - if (!str) return TRUE; - __TRY - { - volatile const WCHAR *p = str; - while (p != str + max) if (!*p++) break; - } - __EXCEPT_SYSCALL - { - return TRUE; - } - __ENDTRY - return FALSE; -} - - /*********************************************************************** * virtual_uninterrupted_read_memory * diff --git a/include/wine/debug.h b/include/wine/debug.h index 912d61a90a..e064aaefb8 100644 --- a/include/wine/debug.h +++ b/include/wine/debug.h @@ -225,7 +225,9 @@ static inline const char *wine_dbgstr_an( const char *str, int n ) if (!str) return "(null)"; if (!((ULONG_PTR)str >> 16)) return wine_dbg_sprintf( "#%04x", LOWORD(str) ); +#ifndef WINE_UNIX_LIB if (IsBadStringPtrA( str, n )) return "(invalid)"; +#endif if (n == -1) for (n = 0; str[n]; n++) ; *dst++ = '"'; while (n-- > 0 && dst <= buffer + sizeof(buffer) - 9) @@ -267,7 +269,9 @@ static inline const char *wine_dbgstr_wn( const WCHAR *str, int n ) if (!str) return "(null)"; if (!((ULONG_PTR)str >> 16)) return wine_dbg_sprintf( "#%04x", LOWORD(str) ); +#ifndef WINE_UNIX_LIB if (IsBadStringPtrW( str, n )) return "(invalid)"; +#endif if (n == -1) for (n = 0; str[n]; n++) ; *dst++ = 'L'; *dst++ = '"';

1 0

Alexandre Julliard : makefiles: Define WINE_UNIX_LIB when building Unix libraries.
by Alexandre Julliard Sept. 18, 2020

Sept. 18, 2020

Module: wine Branch: master Commit: 0b9c7eb295f568ffb1ec856bef2a99727a313736 URL: https://source.winehq.org/git/wine.git/?a=commit;h=0b9c7eb295f568ffb1ec856b… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Sep 18 15:05:44 2020 +0200 makefiles: Define WINE_UNIX_LIB when building Unix libraries. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- tools/makedep.c | 1 + 1 file changed, 1 insertion(+) diff --git a/tools/makedep.c b/tools/makedep.c index b7c26492eb..b3dd03645f 100644 --- a/tools/makedep.c +++ b/tools/makedep.c @@ -2384,6 +2384,7 @@ static struct strarray get_source_defines( struct makefile *make, struct incl_fi strarray_add( &ret, strmake( "-I%s", make->include_paths.str[i] )); strarray_addall( &ret, make->define_args ); strarray_addall( &ret, get_expanded_file_local_var( make, obj, "EXTRADEFS" )); + if ((source->file->flags & FLAG_C_UNIX) && *dll_ext) strarray_add( &ret, "-DWINE_UNIX_LIB" ); return ret; }

1 0

Kevin Puetz : rpcrt4: Fix ARM64 stack corruption in call_server_func.
by Alexandre Julliard Sept. 18, 2020

Sept. 18, 2020

Module: wine Branch: master Commit: e4e7be6bd9e8a70329f5ceb4f6d66f50e3170ae2 URL: https://source.winehq.org/git/wine.git/?a=commit;h=e4e7be6bd9e8a70329f5ceb4… Author: Kevin Puetz <PuetzKevinA(a)JohnDeere.com> Date: Thu Sep 17 19:52:46 2020 -0500 rpcrt4: Fix ARM64 stack corruption in call_server_func. calls with stack_size > 16 pop argument values into x3, overwriting the number of 16-byte stack octwords. This breaks the checks for popping x4..x7 and potentially unbalances sp (based on the vaue of args[16..23]). Use a scratch register (x9) for this count so its lifetime does not conflict with preparing the parameter/result registers. Signed-off-by: Kevin Puetz <PuetzKevinA(a)JohnDeere.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/rpcrt4/ndr_stubless.c | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/dlls/rpcrt4/ndr_stubless.c b/dlls/rpcrt4/ndr_stubless.c index 9f579fcfd4..43debcb91d 100644 --- a/dlls/rpcrt4/ndr_stubless.c +++ b/dlls/rpcrt4/ndr_stubless.c @@ -1180,24 +1180,24 @@ LONG_PTR __cdecl call_server_func(SERVER_ROUTINE func, unsigned char *args, unsi __ASM_GLOBAL_FUNC( call_server_func, "stp x29, x30, [sp, #-16]!\n\t" "mov x29, sp\n\t" - "add x3, x2, #15\n\t" - "lsr x3, x3, #4\n\t" - "sub sp, sp, x3, lsl #4\n\t" + "add x9, x2, #15\n\t" + "lsr x9, x9, #4\n\t" + "sub sp, sp, x9, lsl #4\n\t" "cbz x2, 2f\n" "1:\tsub x2, x2, #8\n\t" "ldr x4, [x1, x2]\n\t" "str x4, [sp, x2]\n\t" "cbnz x2, 1b\n" "2:\tmov x8, x0\n\t" - "cbz x3, 3f\n\t" + "cbz x9, 3f\n\t" "ldp x0, x1, [sp], #16\n\t" - "cmp x3, #1\n\t" + "cmp x9, #1\n\t" "b.le 3f\n\t" "ldp x2, x3, [sp], #16\n\t" - "cmp x3, #2\n\t" + "cmp x9, #2\n\t" "b.le 3f\n\t" "ldp x4, x5, [sp], #16\n\t" - "cmp x3, #3\n\t" + "cmp x9, #3\n\t" "b.le 3f\n\t" "ldp x6, x7, [sp], #16\n" "3:\tblr x8\n\t"

1 0

Kevin Puetz : widl: All VARDESC fields of TKIND_UNION should have oInst=0.
by Alexandre Julliard Sept. 18, 2020

Sept. 18, 2020

Module: wine Branch: master Commit: 44ccc4f638a3b216bddb1bebbe98cae9b3921197 URL: https://source.winehq.org/git/wine.git/?a=commit;h=44ccc4f638a3b216bddb1beb… Author: Kevin Puetz <PuetzKevinA(a)JohnDeere.com> Date: Thu Sep 17 18:24:18 2020 -0500 widl: All VARDESC fields of TKIND_UNION should have oInst=0. Also fix uninitialized value in this field for TKIND_DISPATCH. Signed-off-by: Kevin Puetz <PuetzKevinA(a)JohnDeere.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- tools/widl/write_msft.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tools/widl/write_msft.c b/tools/widl/write_msft.c index 7cfe9e146c..23d1dd8652 100644 --- a/tools/widl/write_msft.c +++ b/tools/widl/write_msft.c @@ -1746,11 +1746,12 @@ static HRESULT add_var_desc(msft_typeinfo_t *typeinfo, UINT index, var_t* var) typeinfo->datawidth += var_datawidth; break; case TKIND_UNION: - typedata[4] = typeinfo->datawidth; + typedata[4] = 0; typeinfo->datawidth = max(typeinfo->datawidth, var_datawidth); break; case TKIND_DISPATCH: var_kind = 3; /* VAR_DISPATCH */ + typedata[4] = 0; typeinfo->datawidth = pointer_size; break; default:

1 0

Isabella Bosia : vkd3d-shader: Print abs(x) as |x|.
by Alexandre Julliard Sept. 18, 2020

Sept. 18, 2020

Module: vkd3d Branch: master Commit: cf02e942eb7a5cf6f53c333ee5f3b34b2e3c1708 URL: https://source.winehq.org/git/vkd3d.git/?a=commit;h=cf02e942eb7a5cf6f53c333… Author: Isabella Bosia <ibosia(a)codeweavers.com> Date: Thu Sep 17 00:14:44 2020 +0100 vkd3d-shader: Print abs(x) as |x|. Signed-off-by: Isabella Bosia <ibosia(a)codeweavers.com> Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- libs/vkd3d-shader/trace.c | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/libs/vkd3d-shader/trace.c b/libs/vkd3d-shader/trace.c index 02abbf9..1e1f819 100644 --- a/libs/vkd3d-shader/trace.c +++ b/libs/vkd3d-shader/trace.c @@ -1033,7 +1033,7 @@ static void shader_dump_src_param(struct vkd3d_string_buffer *buffer, shader_addline(buffer, "!"); if (src_modifier == VKD3DSPSM_ABS || src_modifier == VKD3DSPSM_ABSNEG) - shader_addline(buffer, "abs("); + shader_addline(buffer, "|"); shader_dump_register(buffer, &param->reg, shader_version); @@ -1051,8 +1051,8 @@ static void shader_dump_src_param(struct vkd3d_string_buffer *buffer, case VKD3DSPSM_X2NEG: shader_addline(buffer, "_x2"); break; case VKD3DSPSM_DZ: shader_addline(buffer, "_dz"); break; case VKD3DSPSM_DW: shader_addline(buffer, "_dw"); break; - case VKD3DSPSM_ABSNEG: shader_addline(buffer, ")"); break; - case VKD3DSPSM_ABS: shader_addline(buffer, ")"); break; + case VKD3DSPSM_ABSNEG: + case VKD3DSPSM_ABS: /* handled later */ break; default: shader_addline(buffer, "_unknown_modifier(%#x)", src_modifier); } @@ -1076,6 +1076,8 @@ static void shader_dump_src_param(struct vkd3d_string_buffer *buffer, swizzle_chars[swizzle_z], swizzle_chars[swizzle_w]); } } + if (src_modifier == VKD3DSPSM_ABS || src_modifier == VKD3DSPSM_ABSNEG) + shader_addline(buffer, "|"); } static void shader_dump_ins_modifiers(struct vkd3d_string_buffer *buffer,

1 0

Zebediah Figura : vkd3d-shader: Separate the allocated and used mask in struct vkd3d_shader_signature_element.
by Alexandre Julliard Sept. 18, 2020

Sept. 18, 2020

Module: vkd3d Branch: master Commit: 82c42588e7830b4ec93d707faa45a8263aa44e61 URL: https://source.winehq.org/git/vkd3d.git/?a=commit;h=82c42588e7830b4ec93d707… Author: Zebediah Figura <z.figura12(a)gmail.com> Date: Fri Sep 18 11:38:09 2020 -0500 vkd3d-shader: Separate the allocated and used mask in struct vkd3d_shader_signature_element. Signed-off-by: Zebediah Figura <z.figura12(a)gmail.com> Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- include/vkd3d_shader.h | 9 ++++++++- libs/vkd3d-shader/dxbc.c | 18 +++++++++++++++--- libs/vkd3d-shader/spirv.c | 22 +++++++++++----------- 3 files changed, 34 insertions(+), 15 deletions(-) diff --git a/include/vkd3d_shader.h b/include/vkd3d_shader.h index b048bd7..d7fa3ed 100644 --- a/include/vkd3d_shader.h +++ b/include/vkd3d_shader.h @@ -1074,8 +1074,15 @@ struct vkd3d_shader_signature_element enum vkd3d_shader_component_type component_type; /** Register index. */ unsigned int register_index; - /** Register mask. */ + /** Mask of the register components allocated to this varying. */ unsigned int mask; + /** + * Subset of \ref mask which the shader reads from or writes to. Unlike + * Direct3D shader bytecode, the mask for output and tessellation signatures + * is not inverted, i.e. bits set in this field denote components which are + * written to. + */ + unsigned int used_mask; /** Minimum interpolation precision. */ enum vkd3d_shader_minimum_precision min_precision; }; diff --git a/libs/vkd3d-shader/dxbc.c b/libs/vkd3d-shader/dxbc.c index a62974a..398cd9f 100644 --- a/libs/vkd3d-shader/dxbc.c +++ b/libs/vkd3d-shader/dxbc.c @@ -2139,7 +2139,7 @@ static int shader_parse_signature(DWORD tag, const char *data, DWORD data_size, for (i = 0; i < count; ++i) { - DWORD name_offset; + DWORD name_offset, mask; if (has_stream_index) read_dword(&ptr, &e[i].stream_index); @@ -2157,7 +2157,19 @@ static int shader_parse_signature(DWORD tag, const char *data, DWORD data_size, read_dword(&ptr, &e[i].sysval_semantic); read_dword(&ptr, &e[i].component_type); read_dword(&ptr, &e[i].register_index); - read_dword(&ptr, &e[i].mask); + read_dword(&ptr, &mask); + e[i].mask = mask & 0xff; + e[i].used_mask = (mask >> 8) & 0xff; + switch (tag) + { + case TAG_OSGN: + case TAG_OSG1: + case TAG_OSG5: + case TAG_PCSG: + case TAG_PSG1: + e[i].used_mask = e[i].mask & ~e[i].used_mask; + break; + } if (has_min_precision) read_dword(&ptr, &e[i].min_precision); @@ -2167,7 +2179,7 @@ static int shader_parse_signature(DWORD tag, const char *data, DWORD data_size, TRACE("Stream: %u, semantic: %s, semantic idx: %u, sysval_semantic %#x, " "type %u, register idx: %u, use_mask %#x, input_mask %#x, precision %u.\n", e[i].stream_index, debugstr_a(e[i].semantic_name), e[i].semantic_index, e[i].sysval_semantic, - e[i].component_type, e[i].register_index, (e[i].mask >> 8) & 0xff, e[i].mask & 0xff, e[i].min_precision); + e[i].component_type, e[i].register_index, e[i].used_mask, e[i].mask, e[i].min_precision); } s->elements = e; diff --git a/libs/vkd3d-shader/spirv.c b/libs/vkd3d-shader/spirv.c index ec6987b..fa0d4e0 100644 --- a/libs/vkd3d-shader/spirv.c +++ b/libs/vkd3d-shader/spirv.c @@ -4139,7 +4139,7 @@ static bool needs_private_io_variable(const struct vkd3d_shader_signature *signa if (current->register_index != reg_idx) continue; - write_mask |= current->mask & 0xff; + write_mask |= current->mask; ++count; if (current->sysval_semantic) @@ -4216,7 +4216,7 @@ static uint32_t vkd3d_dxbc_compiler_emit_input(struct vkd3d_dxbc_compiler *compi builtin = get_spirv_builtin_for_sysval(compiler, sysval); - write_mask = signature_element->mask & 0xff; + write_mask = signature_element->mask; component_count = vkd3d_write_mask_component_count(dst->write_mask); if (builtin) @@ -4228,8 +4228,8 @@ static uint32_t vkd3d_dxbc_compiler_emit_input(struct vkd3d_dxbc_compiler *compi else { component_type = signature_element->component_type; - input_component_count = vkd3d_write_mask_component_count(signature_element->mask & 0xff); - component_idx = vkd3d_write_mask_get_component_idx(signature_element->mask & 0xff); + input_component_count = vkd3d_write_mask_component_count(signature_element->mask); + component_idx = vkd3d_write_mask_get_component_idx(signature_element->mask); } if ((use_private_var = builtin && builtin->fixup_pfn)) @@ -4657,10 +4657,10 @@ static void vkd3d_dxbc_compiler_emit_output(struct vkd3d_dxbc_compiler *compiler builtin = vkd3d_get_spirv_builtin(compiler, dst->reg.type, sysval); - write_mask = signature_element->mask & 0xff; + write_mask = signature_element->mask; component_idx = vkd3d_write_mask_get_component_idx(dst->write_mask); - output_component_count = vkd3d_write_mask_component_count(signature_element->mask & 0xff); + output_component_count = vkd3d_write_mask_component_count(signature_element->mask); if (builtin) { component_type = builtin->component_type; @@ -4833,9 +4833,9 @@ static void vkd3d_dxbc_compiler_emit_store_shader_output(struct vkd3d_dxbc_compi unsigned int i, index, array_idx; uint32_t output_id; - dst_write_mask = output->mask & 0xff; + dst_write_mask = output->mask; write_mask &= dst_write_mask; - use_mask = (output->mask >> 8) & 0xff; + use_mask = output->used_mask; if (!write_mask) return; @@ -4847,7 +4847,7 @@ static void vkd3d_dxbc_compiler_emit_store_shader_output(struct vkd3d_dxbc_compi } swizzle = get_shader_output_swizzle(compiler, output->register_index); - uninit_mask = dst_write_mask & use_mask; + uninit_mask = dst_write_mask & ~use_mask; if (uninit_mask) { /* Set values to 0 for not initialized shader output components. */ @@ -6180,7 +6180,7 @@ static void vkd3d_dxbc_compiler_emit_default_control_point_phase(struct vkd3d_dx const struct vkd3d_shader_signature_element *output = &output_signature->elements[i]; const struct vkd3d_shader_signature_element *input = &input_signature->elements[i]; - assert((input->mask & 0xff) == (output->mask & 0xff)); + assert(input->mask == output->mask); assert(input->component_type == output->component_type); if ((input_builtin = get_spirv_builtin_for_sysval(compiler, vkd3d_siv_from_sysval(input->sysval_semantic)))) @@ -6191,7 +6191,7 @@ static void vkd3d_dxbc_compiler_emit_default_control_point_phase(struct vkd3d_dx else { component_type = input->component_type; - component_count = vkd3d_write_mask_component_count(input->mask & 0xff); + component_count = vkd3d_write_mask_component_count(input->mask); } if (input_builtin)

1 0

Zebediah Figura : vkd3d: Print VK_MEMORY_HEAP_MULTI_INSTANCE_BIT in debug_vk_memory_heap_flags().
by Alexandre Julliard Sept. 18, 2020

Sept. 18, 2020

Module: vkd3d Branch: master Commit: 1165ad3f0089e8675f9fa45b5b3cfa6f0ea0738f URL: https://source.winehq.org/git/vkd3d.git/?a=commit;h=1165ad3f0089e8675f9fa45… Author: Zebediah Figura <zfigura(a)codeweavers.com> Date: Thu Sep 17 22:44:22 2020 -0500 vkd3d: Print VK_MEMORY_HEAP_MULTI_INSTANCE_BIT in debug_vk_memory_heap_flags(). Signed-off-by: Zebediah Figura <zfigura(a)codeweavers.com> Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- libs/vkd3d/utils.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libs/vkd3d/utils.c b/libs/vkd3d/utils.c index 7abfd42..f9f2663 100644 --- a/libs/vkd3d/utils.c +++ b/libs/vkd3d/utils.c @@ -699,11 +699,12 @@ const char *debug_vk_queue_flags(VkQueueFlags flags) const char *debug_vk_memory_heap_flags(VkMemoryHeapFlags flags) { - char buffer[50]; + char buffer[80]; buffer[0] = '\0'; #define FLAG_TO_STR(f) if (flags & f) { strcat(buffer, " | "#f); flags &= ~f; } FLAG_TO_STR(VK_MEMORY_HEAP_DEVICE_LOCAL_BIT) + FLAG_TO_STR(VK_MEMORY_HEAP_MULTI_INSTANCE_BIT) #undef FLAG_TO_STR if (flags) FIXME("Unrecognized flag(s) %#x.\n", flags);

1 0

Isabella Bosia : vkd3d-shader: Make shader_dump_data_type print the type 4 times.
by Alexandre Julliard Sept. 18, 2020

Sept. 18, 2020

Module: vkd3d Branch: master Commit: b27eac1cb53db90296e033541c3c5018a88abd5d URL: https://source.winehq.org/git/vkd3d.git/?a=commit;h=b27eac1cb53db90296e0335… Author: Isabella Bosia <ibosia(a)codeweavers.com> Date: Thu Sep 17 00:14:45 2020 +0100 vkd3d-shader: Make shader_dump_data_type print the type 4 times. Signed-off-by: Isabella Bosia <ibosia(a)codeweavers.com> Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- libs/vkd3d-shader/trace.c | 25 ++++++++++++++----------- 1 file changed, 14 insertions(+), 11 deletions(-) diff --git a/libs/vkd3d-shader/trace.c b/libs/vkd3d-shader/trace.c index 1e1f819..587c85c 100644 --- a/libs/vkd3d-shader/trace.c +++ b/libs/vkd3d-shader/trace.c @@ -530,21 +530,24 @@ static void shader_dump_data_type(struct vkd3d_string_buffer *buffer, enum vkd3d { static const char *const data_type_names[] = { - /* VKD3D_DATA_FLOAT */ "(float)", - /* VKD3D_DATA_INT */ "(int)", - /* VKD3D_DATA_RESOURCE */ "(resource)", - /* VKD3D_DATA_SAMPLER */ "(sampler)", - /* VKD3D_DATA_UAV */ "(uav)", - /* VKD3D_DATA_UINT */ "(uint)", - /* VKD3D_DATA_UNORM */ "(unorm)", - /* VKD3D_DATA_SNORM */ "(snorm)", - /* VKD3D_DATA_OPAQUE */ "(opaque)", + /* VKD3D_DATA_FLOAT */ "float", + /* VKD3D_DATA_INT */ "int", + /* VKD3D_DATA_RESOURCE */ "resource", + /* VKD3D_DATA_SAMPLER */ "sampler", + /* VKD3D_DATA_UAV */ "uav", + /* VKD3D_DATA_UINT */ "uint", + /* VKD3D_DATA_UNORM */ "unorm", + /* VKD3D_DATA_SNORM */ "snorm", + /* VKD3D_DATA_OPAQUE */ "opaque", }; + const char *name; if (type <= ARRAY_SIZE(data_type_names)) - shader_addline(buffer, "%s", data_type_names[type]); + name = data_type_names[type]; else - shader_addline(buffer, "(unknown)"); + name = "unknown"; + + shader_addline(buffer, "(%s,%s,%s,%s)", name, name, name, name); } static void shader_dump_decl_usage(struct vkd3d_string_buffer *buffer,

1 0