February 2019 - Wine-commits

Józef Kucia : d3d12/tests: Add test for swapchain and window sizes mismatch.
by Alexandre Julliard 28 Feb '19

28 Feb '19

Module: wine Branch: master Commit: 2aac5c436e6c193882276967f4c2e68f3db6dc81 URL: https://source.winehq.org/git/wine.git/?a=commit;h=2aac5c436e6c193882276967… Author: Józef Kucia <jkucia(a)codeweavers.com> Date: Wed Feb 27 16:04:59 2019 +0100 d3d12/tests: Add test for swapchain and window sizes mismatch. Signed-off-by: Józef Kucia <jkucia(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/d3d12/tests/d3d12.c | 102 +++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 102 insertions(+) diff --git a/dlls/d3d12/tests/d3d12.c b/dlls/d3d12/tests/d3d12.c index 7b610c0..67e7240 100644 --- a/dlls/d3d12/tests/d3d12.c +++ b/dlls/d3d12/tests/d3d12.c @@ -908,6 +908,107 @@ static void test_swapchain_draw(void) destroy_test_context(&context); } +static void test_swapchain_size_mismatch(void) +{ + static const float green[] = {0.0f, 1.0f, 0.0f, 1.0f}; + ID3D12GraphicsCommandList *command_list; + ID3D12Resource *backbuffers[2]; + struct test_context_desc desc; + struct test_context context; + ID3D12Resource *backbuffer; + IDXGISwapChain3 *swapchain; + ID3D12CommandQueue *queue; + unsigned int index, i; + ID3D12Device *device; + ULONG refcount; + HWND window; + HRESULT hr; + RECT rect; + BOOL ret; + + desc.no_pipeline = TRUE; + if (!init_test_context(&context, &desc)) + return; + device = context.device; + command_list = context.list; + queue = context.queue; + + window = CreateWindowA("static", "d3d12_test", WS_VISIBLE, 0, 0, 200, 200, NULL, NULL, NULL, NULL); + swapchain = create_swapchain(queue, window, DXGI_FORMAT_B8G8R8A8_UNORM, 400, 400); + index = IDXGISwapChain3_GetCurrentBackBufferIndex(swapchain); + hr = IDXGISwapChain3_GetBuffer(swapchain, index, &IID_ID3D12Resource, (void **)&backbuffer); + ok(hr == S_OK, "Failed to get swapchain buffer %u, hr %#x.\n", index, hr); + ID3D12Device_CreateRenderTargetView(device, backbuffer, NULL, context.rtv); + + transition_sub_resource_state(command_list, backbuffer, 0, + D3D12_RESOURCE_STATE_PRESENT, D3D12_RESOURCE_STATE_RENDER_TARGET); + ID3D12GraphicsCommandList_ClearRenderTargetView(command_list, context.rtv, green, 0, NULL); + transition_sub_resource_state(command_list, backbuffer, 0, + D3D12_RESOURCE_STATE_RENDER_TARGET, D3D12_RESOURCE_STATE_COPY_SOURCE); + check_sub_resource_uint(backbuffer, 0, queue, command_list, 0xff00ff00, 0); + + reset_command_list(command_list, context.allocator); + transition_sub_resource_state(command_list, backbuffer, 0, + D3D12_RESOURCE_STATE_COPY_SOURCE, D3D12_RESOURCE_STATE_PRESENT); + hr = ID3D12GraphicsCommandList_Close(command_list); + ok(hr == S_OK, "Failed to close command list, hr %#x.\n", hr); + exec_command_list(queue, command_list); + + hr = IDXGISwapChain3_Present(swapchain, 1, 0); + ok(hr == S_OK, "Failed to present, hr %#x.\n", hr); + + wait_queue_idle(device, queue); + reset_command_list(command_list, context.allocator); + + refcount = ID3D12Resource_Release(backbuffer); + ok(!refcount, "Backbuffer has %u references left.\n", refcount); + refcount = IDXGISwapChain3_Release(swapchain); + ok(!refcount, "Swapchain has %u references left.\n", refcount); + DestroyWindow(window); + + window = create_window(WS_VISIBLE); + ret = GetClientRect(window, &rect); + ok(ret, "Failed to get client rect.\n"); + swapchain = create_swapchain(queue, window, DXGI_FORMAT_B8G8R8A8_UNORM, rect.right, rect.bottom); + + for (i = 0; i < ARRAY_SIZE(backbuffers); ++i) + { + hr = IDXGISwapChain3_GetBuffer(swapchain, i, &IID_ID3D12Resource, (void **)&backbuffers[i]); + ok(hr == S_OK, "Failed to get swapchain buffer %u, hr %#x.\n", i, hr); + } + + for (i = 0; i < 20; ++i) + { + index = IDXGISwapChain3_GetCurrentBackBufferIndex(swapchain); + ID3D12Device_CreateRenderTargetView(device, backbuffers[index], NULL, context.rtv); + + transition_sub_resource_state(command_list, backbuffers[index], 0, + D3D12_RESOURCE_STATE_PRESENT, D3D12_RESOURCE_STATE_RENDER_TARGET); + ID3D12GraphicsCommandList_ClearRenderTargetView(command_list, context.rtv, green, 0, NULL); + transition_sub_resource_state(command_list, backbuffers[index], 0, + D3D12_RESOURCE_STATE_RENDER_TARGET, D3D12_RESOURCE_STATE_PRESENT); + hr = ID3D12GraphicsCommandList_Close(command_list); + ok(hr == S_OK, "Failed to close command list, hr %#x.\n", hr); + exec_command_list(queue, command_list); + + hr = IDXGISwapChain3_Present(swapchain, 1, 0); + ok(hr == S_OK, "Failed to present, hr %#x.\n", hr); + + wait_queue_idle(device, queue); + reset_command_list(command_list, context.allocator); + + if (i == 6) + MoveWindow(window, 0, 0, 100, 100, TRUE); + } + + for (i = 0; i < ARRAY_SIZE(backbuffers); ++i) + ID3D12Resource_Release(backbuffers[i]); + refcount = IDXGISwapChain3_Release(swapchain); + ok(!refcount, "Swapchain has %u references left.\n", refcount); + DestroyWindow(window); + destroy_test_context(&context); +} + START_TEST(d3d12) { BOOL enable_debug_layer = FALSE; @@ -937,4 +1038,5 @@ START_TEST(d3d12) test_interfaces(); test_draw(); test_swapchain_draw(); + test_swapchain_size_mismatch(); }

1 0

Józef Kucia : dxgi: Implement 0 sync interval for D3D12 swapchains.
by Alexandre Julliard 28 Feb '19

28 Feb '19

Module: wine Branch: master Commit: b78f02922d392de34c1a060788c245a336d699ce URL: https://source.winehq.org/git/wine.git/?a=commit;h=b78f02922d392de34c1a0607… Author: Józef Kucia <jkucia(a)codeweavers.com> Date: Wed Feb 27 16:05:00 2019 +0100 dxgi: Implement 0 sync interval for D3D12 swapchains. Signed-off-by: Józef Kucia <jkucia(a)codeweavers.com> Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/dxgi/swapchain.c | 101 ++++++++++++++++++++++++++++++++++++++++++++++++-- 1 file changed, 98 insertions(+), 3 deletions(-) diff --git a/dlls/dxgi/swapchain.c b/dlls/dxgi/swapchain.c index 06fb7ef..fb55820 100644 --- a/dlls/dxgi/swapchain.c +++ b/dlls/dxgi/swapchain.c @@ -892,6 +892,7 @@ struct d3d12_swapchain unsigned int buffer_count; unsigned int vk_swapchain_width; unsigned int vk_swapchain_height; + VkPresentModeKHR present_mode; uint32_t current_buffer_index; struct dxgi_vk_funcs vk_funcs; @@ -1021,6 +1022,51 @@ static HRESULT vk_select_memory_type(const struct dxgi_vk_funcs *vk_funcs, return E_FAIL; } +static BOOL d3d12_swapchain_is_present_mode_supported(struct d3d12_swapchain *swapchain, + VkPresentModeKHR present_mode) +{ + VkPhysicalDevice vk_physical_device = swapchain->vk_physical_device; + const struct dxgi_vk_funcs *vk_funcs = &swapchain->vk_funcs; + VkPresentModeKHR *modes; + uint32_t count, i; + BOOL supported; + VkResult vr; + + if (present_mode == VK_PRESENT_MODE_FIFO_KHR) + return TRUE; + + if ((vr = vk_funcs->p_vkGetPhysicalDeviceSurfacePresentModesKHR(vk_physical_device, + swapchain->vk_surface, &count, NULL)) < 0) + { + WARN("Failed to get count of available present modes, vr %d.\n", vr); + return FALSE; + } + + supported = FALSE; + + if (!(modes = heap_calloc(count, sizeof(*modes)))) + return FALSE; + if ((vr = vk_funcs->p_vkGetPhysicalDeviceSurfacePresentModesKHR(vk_physical_device, + swapchain->vk_surface, &count, modes)) >= 0) + { + for (i = 0; i < count; ++i) + { + if (modes[i] == present_mode) + { + supported = TRUE; + break; + } + } + } + else + { + WARN("Failed to get available present modes, vr %d.\n", vr); + } + heap_free(modes); + + return supported; +} + static HRESULT d3d12_swapchain_create_user_buffers(struct d3d12_swapchain *swapchain, VkFormat vk_format) { const struct dxgi_vk_funcs *vk_funcs = &swapchain->vk_funcs; @@ -1527,7 +1573,7 @@ static HRESULT d3d12_swapchain_create_vulkan_swapchain(struct d3d12_swapchain *s vk_swapchain_desc.pQueueFamilyIndices = NULL; vk_swapchain_desc.preTransform = surface_caps.currentTransform; vk_swapchain_desc.compositeAlpha = VK_COMPOSITE_ALPHA_OPAQUE_BIT_KHR; - vk_swapchain_desc.presentMode = VK_PRESENT_MODE_FIFO_KHR; + vk_swapchain_desc.presentMode = swapchain->present_mode; vk_swapchain_desc.clipped = VK_TRUE; vk_swapchain_desc.oldSwapchain = swapchain->vk_swapchain; if ((vr = vk_funcs->p_vkCreateSwapchainKHR(vk_device, &vk_swapchain_desc, NULL, &vk_swapchain)) < 0) @@ -1922,6 +1968,52 @@ static HRESULT STDMETHODCALLTYPE d3d12_swapchain_GetCoreWindow(IDXGISwapChain3 * return DXGI_ERROR_INVALID_CALL; } +static HRESULT d3d12_swapchain_set_sync_interval(struct d3d12_swapchain *swapchain, + unsigned int sync_interval) +{ + VkPresentModeKHR present_mode; + HRESULT hr; + + switch (sync_interval) + { + case 0: + present_mode = VK_PRESENT_MODE_IMMEDIATE_KHR; + break; + default: + FIXME("Unsupported sync interval %u.\n", sync_interval); + case 1: + present_mode = VK_PRESENT_MODE_FIFO_KHR; + break; + } + + if (swapchain->present_mode == present_mode) + return S_OK; + + if (!swapchain->vk_images[swapchain->current_buffer_index]) + { + FIXME("Cannot recreate swapchain without user images.\n"); + return S_OK; + } + + if (!d3d12_swapchain_is_present_mode_supported(swapchain, present_mode)) + { + FIXME("Vulkan present mode %#x is not supported.\n", present_mode); + return S_OK; + } + + d3d12_swapchain_destroy_buffers(swapchain, FALSE); + + swapchain->present_mode = present_mode; + + if (FAILED(hr = d3d12_swapchain_create_vulkan_swapchain(swapchain))) + { + ERR("Failed to recreate Vulkan swapchain, hr %#x.\n", hr); + return hr; + } + + return d3d12_swapchain_acquire_next_image(swapchain); +} + static VkResult d3d12_swapchain_blit_buffer(struct d3d12_swapchain *swapchain, VkQueue vk_queue) { const struct dxgi_vk_funcs *vk_funcs = &swapchain->vk_funcs; @@ -1989,8 +2081,6 @@ static HRESULT STDMETHODCALLTYPE d3d12_swapchain_Present1(IDXGISwapChain3 *iface WARN("Invalid sync interval %u.\n", sync_interval); return DXGI_ERROR_INVALID_CALL; } - if (sync_interval != 1) - FIXME("Ignoring sync interval %u.\n", sync_interval); if (flags & ~DXGI_PRESENT_TEST) FIXME("Unimplemented flags %#x.\n", flags); @@ -2003,6 +2093,9 @@ static HRESULT STDMETHODCALLTYPE d3d12_swapchain_Present1(IDXGISwapChain3 *iface if (present_parameters) FIXME("Ignored present parameters %p.\n", present_parameters); + if (FAILED(hr = d3d12_swapchain_set_sync_interval(swapchain, sync_interval))) + return hr; + if (!(vk_queue = vkd3d_acquire_vk_queue(swapchain->command_queue))) { ERR("Failed to acquire Vulkan queue.\n"); @@ -2391,6 +2484,8 @@ static HRESULT d3d12_swapchain_init(struct d3d12_swapchain *swapchain, IWineDXGI swapchain->desc = *swapchain_desc; swapchain->fullscreen_desc = *fullscreen_desc; + swapchain->present_mode = VK_PRESENT_MODE_FIFO_KHR; + switch (swapchain_desc->SwapEffect) { case DXGI_SWAP_EFFECT_FLIP_SEQUENTIAL:

1 0

Józef Kucia : dxgi: Recreate Vulkan swapchain only when current buffer index is 0.
by Alexandre Julliard 28 Feb '19

28 Feb '19

Module: wine Branch: master Commit: 0510bca6964a103c5d38d1f5b5eb2a2a72d6a2b9 URL: https://source.winehq.org/git/wine.git/?a=commit;h=0510bca6964a103c5d38d1f5… Author: Józef Kucia <jkucia(a)codeweavers.com> Date: Wed Feb 27 16:05:01 2019 +0100 dxgi: Recreate Vulkan swapchain only when current buffer index is 0. Signed-off-by: Józef Kucia <jkucia(a)codeweavers.com> Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/dxgi/swapchain.c | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/dlls/dxgi/swapchain.c b/dlls/dxgi/swapchain.c index fb55820..3c09516 100644 --- a/dlls/dxgi/swapchain.c +++ b/dlls/dxgi/swapchain.c @@ -1989,6 +1989,16 @@ static HRESULT d3d12_swapchain_set_sync_interval(struct d3d12_swapchain *swapcha if (swapchain->present_mode == present_mode) return S_OK; + /* + * We recreate the swapchain only when the current buffer index is 0, in order to preserve the + * expected back buffer index sequence. + */ + if (swapchain->current_buffer_index) + { + WARN("Skipping sync interval change, current buffer index %u.\n", swapchain->current_buffer_index); + return S_OK; + } + if (!swapchain->vk_images[swapchain->current_buffer_index]) { FIXME("Cannot recreate swapchain without user images.\n");

1 0

Józef Kucia : d3d12/tests: Add test for backbuffer index after changing sync interval.
by Alexandre Julliard 28 Feb '19

28 Feb '19

Module: wine Branch: master Commit: f451c7b586f7bf390e34ab2e73e383b97cb6f2ca URL: https://source.winehq.org/git/wine.git/?a=commit;h=f451c7b586f7bf390e34ab2e… Author: Józef Kucia <jkucia(a)codeweavers.com> Date: Wed Feb 27 16:05:02 2019 +0100 d3d12/tests: Add test for backbuffer index after changing sync interval. Signed-off-by: Józef Kucia <jkucia(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/d3d12/tests/d3d12.c | 77 ++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 77 insertions(+) diff --git a/dlls/d3d12/tests/d3d12.c b/dlls/d3d12/tests/d3d12.c index 67e7240..dec7cf8 100644 --- a/dlls/d3d12/tests/d3d12.c +++ b/dlls/d3d12/tests/d3d12.c @@ -1009,6 +1009,82 @@ static void test_swapchain_size_mismatch(void) destroy_test_context(&context); } +static void test_swapchain_backbuffer_index(void) +{ + static const float green[] = {0.0f, 1.0f, 0.0f, 1.0f}; + unsigned int previous_index, expected_index, index, i; + ID3D12GraphicsCommandList *command_list; + ID3D12Resource *backbuffers[2]; + struct test_context_desc desc; + struct test_context context; + unsigned int sync_interval; + IDXGISwapChain3 *swapchain; + ID3D12CommandQueue *queue; + ID3D12Device *device; + ULONG refcount; + HWND window; + HRESULT hr; + RECT rect; + BOOL ret; + + desc.no_pipeline = TRUE; + if (!init_test_context(&context, &desc)) + return; + device = context.device; + command_list = context.list; + queue = context.queue; + + window = create_window(WS_VISIBLE); + ret = GetClientRect(window, &rect); + ok(ret, "Failed to get client rect.\n"); + swapchain = create_swapchain(queue, window, DXGI_FORMAT_B8G8R8A8_UNORM, rect.right, rect.bottom); + + for (i = 0; i < ARRAY_SIZE(backbuffers); ++i) + { + hr = IDXGISwapChain3_GetBuffer(swapchain, i, &IID_ID3D12Resource, (void **)&backbuffers[i]); + ok(hr == S_OK, "Failed to get swapchain buffer %u, hr %#x.\n", i, hr); + } + + previous_index = 1; + for (i = 0; i < 20; ++i) + { + index = IDXGISwapChain3_GetCurrentBackBufferIndex(swapchain); + ID3D12Device_CreateRenderTargetView(device, backbuffers[index], NULL, context.rtv); + + expected_index = (previous_index + 1) % 2; + ok(index == expected_index, "Test %u: Got index %u, expected %u.\n", i, index, expected_index); + + transition_sub_resource_state(command_list, backbuffers[index], 0, + D3D12_RESOURCE_STATE_PRESENT, D3D12_RESOURCE_STATE_RENDER_TARGET); + ID3D12GraphicsCommandList_ClearRenderTargetView(command_list, context.rtv, green, 0, NULL); + transition_sub_resource_state(command_list, backbuffers[index], 0, + D3D12_RESOURCE_STATE_RENDER_TARGET, D3D12_RESOURCE_STATE_PRESENT); + hr = ID3D12GraphicsCommandList_Close(command_list); + ok(hr == S_OK, "Failed to close command list, hr %#x.\n", hr); + exec_command_list(queue, command_list); + + if (i <= 4 || (8 <= i && i <= 14)) + sync_interval = 1; + else + sync_interval = 0; + + hr = IDXGISwapChain3_Present(swapchain, sync_interval, 0); + ok(hr == S_OK, "Failed to present, hr %#x.\n", hr); + + wait_queue_idle(device, queue); + reset_command_list(command_list, context.allocator); + + previous_index = index; + } + + for (i = 0; i < ARRAY_SIZE(backbuffers); ++i) + ID3D12Resource_Release(backbuffers[i]); + refcount = IDXGISwapChain3_Release(swapchain); + ok(!refcount, "Swapchain has %u references left.\n", refcount); + DestroyWindow(window); + destroy_test_context(&context); +} + START_TEST(d3d12) { BOOL enable_debug_layer = FALSE; @@ -1039,4 +1115,5 @@ START_TEST(d3d12) test_draw(); test_swapchain_draw(); test_swapchain_size_mismatch(); + test_swapchain_backbuffer_index(); }

1 0

Józef Kucia : wined3d: Rename ShaderBackend registry setting to shader_backend.
by Alexandre Julliard 28 Feb '19

28 Feb '19

Module: wine Branch: master Commit: 660b446aaeca506499a6975f8dd71a9d6fa0b893 URL: https://source.winehq.org/git/wine.git/?a=commit;h=660b446aaeca506499a6975f… Author: Józef Kucia <jkucia(a)codeweavers.com> Date: Wed Feb 27 16:05:03 2019 +0100 wined3d: Rename ShaderBackend registry setting to shader_backend. This appears to be preferred naming convention for new registry keys. Signed-off-by: Józef Kucia <jkucia(a)codeweavers.com> Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wined3d/wined3d_main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/wined3d/wined3d_main.c b/dlls/wined3d/wined3d_main.c index fb70b94..5ca58ae 100644 --- a/dlls/wined3d/wined3d_main.c +++ b/dlls/wined3d/wined3d_main.c @@ -219,7 +219,7 @@ static BOOL wined3d_dll_init(HINSTANCE hInstDLL) tmpvalue >> 16, tmpvalue & 0xffff); wined3d_settings.max_gl_version = tmpvalue; } - if (!get_config_key(hkey, appkey, "ShaderBackend", buffer, size)) + if (!get_config_key(hkey, appkey, "shader_backend", buffer, size)) { if (!strcasecmp(buffer, "glsl")) {

1 0

Józef Kucia : wined3d: Add GPU information for Intel HD Graphics 620.
by Alexandre Julliard 28 Feb '19

28 Feb '19

Module: wine Branch: master Commit: 88aaee78fe96084f2d1ff5c224b04d10c459088e URL: https://source.winehq.org/git/wine.git/?a=commit;h=88aaee78fe96084f2d1ff5c2… Author: Józef Kucia <jkucia(a)codeweavers.com> Date: Wed Feb 27 16:39:42 2019 +0100 wined3d: Add GPU information for Intel HD Graphics 620. Signed-off-by: Józef Kucia <jkucia(a)codeweavers.com> Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wined3d/directx.c | 1 + dlls/wined3d/wined3d_private.h | 1 + 2 files changed, 2 insertions(+) diff --git a/dlls/wined3d/directx.c b/dlls/wined3d/directx.c index 0c70f04..a05f6cc 100644 --- a/dlls/wined3d/directx.c +++ b/dlls/wined3d/directx.c @@ -456,6 +456,7 @@ static const struct wined3d_gpu_description gpu_description_table[] = {HW_VENDOR_INTEL, CARD_INTEL_IP580_2, "Intel(R) Iris(TM) Pro Graphics 580", DRIVER_INTEL_HD4000, 2048}, {HW_VENDOR_INTEL, CARD_INTEL_IPP580_1, "Intel(R) Iris(TM) Pro Graphics P580", DRIVER_INTEL_HD4000, 2048}, {HW_VENDOR_INTEL, CARD_INTEL_IPP580_2, "Intel(R) Iris(TM) Pro Graphics P580", DRIVER_INTEL_HD4000, 2048}, + {HW_VENDOR_INTEL, CARD_INTEL_HD620, "Intel(R) HD Graphics 620", DRIVER_INTEL_HD4000, 3072}, {HW_VENDOR_INTEL, CARD_INTEL_HD630, "Intel(R) HD Graphics 630", DRIVER_INTEL_HD4000, 3072}, }; diff --git a/dlls/wined3d/wined3d_private.h b/dlls/wined3d/wined3d_private.h index f2b9b5f..1b70582 100644 --- a/dlls/wined3d/wined3d_private.h +++ b/dlls/wined3d/wined3d_private.h @@ -2520,6 +2520,7 @@ enum wined3d_pci_device CARD_INTEL_IP580_2 = 0x193b, CARD_INTEL_IPP580_1 = 0x193a, CARD_INTEL_IPP580_2 = 0x193d, + CARD_INTEL_HD620 = 0x5916, CARD_INTEL_HD630 = 0x5912, };

1 0

Alexandre Julliard : winebuild: Make thiscall an entry point flag instead of a function type.
by Alexandre Julliard 28 Feb '19

28 Feb '19

Module: wine Branch: master Commit: 28debd82fc90e551e8fa8fbee8858df237cb3301 URL: https://source.winehq.org/git/wine.git/?a=commit;h=28debd82fc90e551e8fa8fbe… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Feb 27 17:09:51 2019 +0100 winebuild: Make thiscall an entry point flag instead of a function type. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- tools/winebuild/build.h | 2 +- tools/winebuild/import.c | 2 -- tools/winebuild/parser.c | 40 +++++++++++++++++++++++++++------------- tools/winebuild/spec32.c | 12 +++++------- tools/winebuild/winebuild.man.in | 11 ++++++----- 5 files changed, 39 insertions(+), 28 deletions(-) diff --git a/tools/winebuild/build.h b/tools/winebuild/build.h index fbc2cd9..15d22a0 100644 --- a/tools/winebuild/build.h +++ b/tools/winebuild/build.h @@ -47,7 +47,6 @@ typedef enum TYPE_STDCALL, /* stdcall function (Win32) */ TYPE_CDECL, /* cdecl function (Win32) */ TYPE_VARARGS, /* varargs function (Win32) */ - TYPE_THISCALL, /* thiscall function (Win32 on i386) */ TYPE_EXTERN, /* external symbol (Win32) */ TYPE_NBTYPES } ORD_TYPE; @@ -173,6 +172,7 @@ struct strarray #define FLAG_REGISTER 0x10 /* use register calling convention */ #define FLAG_PRIVATE 0x20 /* function is private (cannot be imported) */ #define FLAG_ORDINAL 0x40 /* function should be imported by ordinal */ +#define FLAG_THISCALL 0x80 /* use thiscall calling convention */ #define FLAG_FORWARD 0x100 /* function is a forwarded name */ #define FLAG_EXT_LINK 0x200 /* function links to an external symbol */ diff --git a/tools/winebuild/import.c b/tools/winebuild/import.c index 5e5866c..893691e 100644 --- a/tools/winebuild/import.c +++ b/tools/winebuild/import.c @@ -466,7 +466,6 @@ static void check_undefined_exports( DLLSPEC *spec ) case TYPE_STDCALL: case TYPE_CDECL: case TYPE_VARARGS: - case TYPE_THISCALL: if (link_ext_symbols) { odp->flags |= FLAG_EXT_LINK; @@ -1444,7 +1443,6 @@ static void build_unix_import_lib( DLLSPEC *spec ) case TYPE_VARARGS: case TYPE_CDECL: case TYPE_STDCALL: - case TYPE_THISCALL: prefix = (!odp->name || (odp->flags & FLAG_ORDINAL)) ? import_ord_prefix : import_func_prefix; new_output_as_file( spec->file_name ); output( "\t.text\n" ); diff --git a/tools/winebuild/parser.c b/tools/winebuild/parser.c index ad99654..11b05aa 100644 --- a/tools/winebuild/parser.c +++ b/tools/winebuild/parser.c @@ -56,7 +56,6 @@ static const char * const TypeNames[TYPE_NBTYPES] = "stdcall", /* TYPE_STDCALL */ "cdecl", /* TYPE_CDECL */ "varargs", /* TYPE_VARARGS */ - "thiscall", /* TYPE_THISCALL */ "extern" /* TYPE_EXTERN */ }; @@ -69,6 +68,7 @@ static const char * const FlagNames[] = "register", /* FLAG_REGISTER */ "private", /* FLAG_PRIVATE */ "ordinal", /* FLAG_ORDINAL */ + "thiscall", /* FLAG_THISCALL */ NULL }; @@ -287,10 +287,18 @@ static int parse_spec_arguments( ORDDEF *odp, DLLSPEC *spec, int optional ) } odp->u.func.nb_args = i; - if (odp->type == TYPE_THISCALL && (!i || odp->u.func.args[0] != ARG_PTR)) + if (odp->flags & FLAG_THISCALL) { - error( "First argument of a thiscall function must be a pointer\n" ); - return 0; + if (odp->type != TYPE_STDCALL) + { + error( "A thiscall function must use the stdcall convention\n" ); + return 0; + } + if (!i || odp->u.func.args[0] != ARG_PTR) + { + error( "First argument of a thiscall function must be a pointer\n" ); + return 0; + } } return 1; } @@ -311,11 +319,6 @@ static int parse_spec_export( ORDDEF *odp, DLLSPEC *spec ) error( "'stdcall' not supported for Win16\n" ); return 0; } - if (!is_win32 && odp->type == TYPE_THISCALL) - { - error( "'thiscall' not supported for Win16\n" ); - return 0; - } if (is_win32 && odp->type == TYPE_PASCAL) { error( "'pascal' not supported for Win32\n" ); @@ -327,6 +330,9 @@ static int parse_spec_export( ORDDEF *odp, DLLSPEC *spec ) if (odp->type == TYPE_VARARGS) odp->flags |= FLAG_NORELAY; /* no relay debug possible for varags entry point */ + if (target_cpu != CPU_x86) + odp->flags &= ~FLAG_THISCALL; + if (!(token = GetToken(1))) { if (!strcmp( odp->name, "@" )) @@ -349,7 +355,7 @@ static int parse_spec_export( ORDDEF *odp, DLLSPEC *spec ) odp->flags |= FLAG_FORWARD; } } - if (target_cpu == CPU_x86 && odp->type == TYPE_THISCALL && !(odp->flags & FLAG_FORWARD)) + if ((odp->flags & FLAG_THISCALL) && !(odp->flags & FLAG_FORWARD)) { char *link_name = strmake( "__thiscall_%s", odp->link_name ); free( odp->link_name ); @@ -506,6 +512,7 @@ static const char *parse_spec_flags( DLLSPEC *spec, ORDDEF *odp ) error( "Flag '%s' is not supported in Win32\n", FlagNames[i] ); break; case FLAG_RET64: + case FLAG_THISCALL: if (spec->type == SPEC_WIN16) error( "Flag '%s' is not supported in Win16\n", FlagNames[i] ); break; @@ -538,8 +545,16 @@ static int parse_spec_ordinal( int ordinal, DLLSPEC *spec ) if (odp->type >= TYPE_NBTYPES) { - error( "Expected type after ordinal, found '%s' instead\n", token ); - goto error; + if (!strcmp( token, "thiscall" )) /* for backwards compatibility */ + { + odp->type = TYPE_STDCALL; + odp->flags |= FLAG_THISCALL; + } + else + { + error( "Expected type after ordinal, found '%s' instead\n", token ); + goto error; + } } if (!(token = GetToken(0))) goto error; @@ -571,7 +586,6 @@ static int parse_spec_ordinal( int ordinal, DLLSPEC *spec ) case TYPE_STDCALL: case TYPE_VARARGS: case TYPE_CDECL: - case TYPE_THISCALL: if (!parse_spec_export( odp, spec )) goto error; break; case TYPE_ABS: diff --git a/tools/winebuild/spec32.c b/tools/winebuild/spec32.c index 3552eca..1b4e534 100644 --- a/tools/winebuild/spec32.c +++ b/tools/winebuild/spec32.c @@ -58,7 +58,6 @@ static inline int needs_relay( const ORDDEF *odp ) { case TYPE_STDCALL: case TYPE_CDECL: - case TYPE_THISCALL: break; case TYPE_STUB: if (odp->u.func.nb_args != -1) break; @@ -129,7 +128,7 @@ static void get_arg_string( ORDDEF *odp, char str[MAX_ARGUMENTS + 1] ) break; } } - if (target_cpu == CPU_x86 && odp->type == TYPE_THISCALL) str[0] = 't'; + if (odp->flags & FLAG_THISCALL) str[0] = 't'; /* append return value */ if (get_ptr_size() == 4 && (odp->flags & FLAG_RET64)) @@ -225,7 +224,7 @@ static void output_relay_debug( DLLSPEC *spec ) switch (target_cpu) { case CPU_x86: - if (odp->type == TYPE_THISCALL) /* add the this pointer */ + if (odp->flags & FLAG_THISCALL) /* add the this pointer */ { output( "\tpopl %%eax\n" ); output( "\tpushl %%ecx\n" ); @@ -246,7 +245,7 @@ static void output_relay_debug( DLLSPEC *spec ) output( "\tcall *4(%%eax)\n" ); output_cfi( ".cfi_adjust_cfa_offset -8" ); - if (odp->type == TYPE_STDCALL || odp->type == TYPE_THISCALL) + if (odp->type == TYPE_STDCALL) output( "\tret $%u\n", get_args_size( odp )); else output( "\tret\n" ); @@ -392,7 +391,6 @@ void output_exports( DLLSPEC *spec ) case TYPE_STDCALL: case TYPE_VARARGS: case TYPE_CDECL: - case TYPE_THISCALL: if (odp->flags & FLAG_FORWARD) { output( "\t%s .L__wine_spec_forwards+%u\n", get_asm_ptr_keyword(), fwd_size ); @@ -953,7 +951,6 @@ void output_def_file( DLLSPEC *spec, int include_private ) /* fall through */ case TYPE_VARARGS: case TYPE_CDECL: - case TYPE_THISCALL: /* try to reduce output */ if(strcmp(name, odp->link_name) || (odp->flags & FLAG_FORWARD)) output( "=%s", odp->link_name ); @@ -969,7 +966,8 @@ void output_def_file( DLLSPEC *spec, int include_private ) else if (strcmp(name, odp->link_name)) /* try to reduce output */ { output( "=%s", odp->link_name ); - if (!kill_at && target_cpu == CPU_x86) output( "@%d", at_param ); + if (!kill_at && target_cpu == CPU_x86 && !(odp->flags & FLAG_THISCALL)) + output( "@%d", at_param ); } break; } diff --git a/tools/winebuild/winebuild.man.in b/tools/winebuild/winebuild.man.in index 51ad544..4455e63 100644 --- a/tools/winebuild/winebuild.man.in +++ b/tools/winebuild/winebuild.man.in @@ -299,6 +299,12 @@ accessed through GetProcAddress. The entry point will be imported by ordinal instead of by name. The name is still exported. .TP +.B -thiscall +The function uses the +.I thiscall +calling convention (first parameter in %ecx register on i386). +.TP +.RE .BI -arch= cpu\fR[\fB,\fIcpu\fR] The entry point is only available on the specified CPU architecture(s). The names \fBwin32\fR and \fBwin64\fR match all @@ -335,11 +341,6 @@ for a Win16 or Win32 function using the C calling convention .B varargs for a Win16 or Win32 function using the C calling convention with a variable number of arguments -.TP -.B thiscall -for a Win32 function using the -.I thiscall -calling convention (first parameter in %ecx register on i386) .RE .PP .I args

1 0

Alexandre Julliard : winebuild: Add support for fastcall entry points.
by Alexandre Julliard 28 Feb '19

28 Feb '19

Module: wine Branch: master Commit: 84a7a7ea0769d6dd833cd6d1d4f2502996d92690 URL: https://source.winehq.org/git/wine.git/?a=commit;h=84a7a7ea0769d6dd833cd6d1… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Feb 27 17:11:08 2019 +0100 winebuild: Add support for fastcall entry points. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- tools/winebuild/build.h | 7 ++++--- tools/winebuild/parser.c | 27 ++++++++++++++++++++++++--- tools/winebuild/spec32.c | 8 +++++--- tools/winebuild/winebuild.man.in | 6 ++++++ 4 files changed, 39 insertions(+), 9 deletions(-) diff --git a/tools/winebuild/build.h b/tools/winebuild/build.h index 15d22a0..eee98eb 100644 --- a/tools/winebuild/build.h +++ b/tools/winebuild/build.h @@ -173,10 +173,11 @@ struct strarray #define FLAG_PRIVATE 0x20 /* function is private (cannot be imported) */ #define FLAG_ORDINAL 0x40 /* function should be imported by ordinal */ #define FLAG_THISCALL 0x80 /* use thiscall calling convention */ +#define FLAG_FASTCALL 0x100 /* use fastcall calling convention */ -#define FLAG_FORWARD 0x100 /* function is a forwarded name */ -#define FLAG_EXT_LINK 0x200 /* function links to an external symbol */ -#define FLAG_EXPORT32 0x400 /* 32-bit export in 16-bit spec file */ +#define FLAG_FORWARD 0x200 /* function is a forwarded name */ +#define FLAG_EXT_LINK 0x400 /* function links to an external symbol */ +#define FLAG_EXPORT32 0x800 /* 32-bit export in 16-bit spec file */ #define FLAG_CPU(cpu) (0x01000 << (cpu)) #define FLAG_CPU_MASK (FLAG_CPU(CPU_LAST + 1) - FLAG_CPU(0)) diff --git a/tools/winebuild/parser.c b/tools/winebuild/parser.c index 11b05aa..dfb7f6f 100644 --- a/tools/winebuild/parser.c +++ b/tools/winebuild/parser.c @@ -69,6 +69,7 @@ static const char * const FlagNames[] = "private", /* FLAG_PRIVATE */ "ordinal", /* FLAG_ORDINAL */ "thiscall", /* FLAG_THISCALL */ + "fastcall", /* FLAG_FASTCALL */ NULL }; @@ -300,6 +301,24 @@ static int parse_spec_arguments( ORDDEF *odp, DLLSPEC *spec, int optional ) return 0; } } + if (odp->flags & FLAG_FASTCALL) + { + if (odp->type != TYPE_STDCALL) + { + error( "A fastcall function must use the stdcall convention\n" ); + return 0; + } + if (!i || (odp->u.func.args[0] != ARG_PTR && odp->u.func.args[0] != ARG_LONG)) + { + error( "First argument of a fastcall function must be a pointer or integer\n" ); + return 0; + } + if (i > 1 && odp->u.func.args[1] != ARG_PTR && odp->u.func.args[1] != ARG_LONG) + { + error( "Second argument of a fastcall function must be a pointer or integer\n" ); + return 0; + } + } return 1; } @@ -331,7 +350,7 @@ static int parse_spec_export( ORDDEF *odp, DLLSPEC *spec ) odp->flags |= FLAG_NORELAY; /* no relay debug possible for varags entry point */ if (target_cpu != CPU_x86) - odp->flags &= ~FLAG_THISCALL; + odp->flags &= ~(FLAG_THISCALL | FLAG_FASTCALL); if (!(token = GetToken(1))) { @@ -355,9 +374,10 @@ static int parse_spec_export( ORDDEF *odp, DLLSPEC *spec ) odp->flags |= FLAG_FORWARD; } } - if ((odp->flags & FLAG_THISCALL) && !(odp->flags & FLAG_FORWARD)) + if ((odp->flags & (FLAG_THISCALL | FLAG_FASTCALL)) && !(odp->flags & FLAG_FORWARD)) { - char *link_name = strmake( "__thiscall_%s", odp->link_name ); + char *link_name = strmake( "__%s_%s", (odp->flags & FLAG_THISCALL) ? "thiscall" : "fastcall", + odp->link_name ); free( odp->link_name ); odp->link_name = link_name; } @@ -513,6 +533,7 @@ static const char *parse_spec_flags( DLLSPEC *spec, ORDDEF *odp ) break; case FLAG_RET64: case FLAG_THISCALL: + case FLAG_FASTCALL: if (spec->type == SPEC_WIN16) error( "Flag '%s' is not supported in Win16\n", FlagNames[i] ); break; diff --git a/tools/winebuild/spec32.c b/tools/winebuild/spec32.c index 1b4e534..44d1c2c 100644 --- a/tools/winebuild/spec32.c +++ b/tools/winebuild/spec32.c @@ -128,7 +128,8 @@ static void get_arg_string( ORDDEF *odp, char str[MAX_ARGUMENTS + 1] ) break; } } - if (odp->flags & FLAG_THISCALL) str[0] = 't'; + if (odp->flags & (FLAG_THISCALL | FLAG_FASTCALL)) str[0] = 't'; + if ((odp->flags & FLAG_FASTCALL) && odp->u.func.nb_args > 1) str[1] = 't'; /* append return value */ if (get_ptr_size() == 4 && (odp->flags & FLAG_RET64)) @@ -224,9 +225,10 @@ static void output_relay_debug( DLLSPEC *spec ) switch (target_cpu) { case CPU_x86: - if (odp->flags & FLAG_THISCALL) /* add the this pointer */ + if (odp->flags & (FLAG_THISCALL | FLAG_FASTCALL)) /* add the register arguments */ { output( "\tpopl %%eax\n" ); + if ((odp->flags & FLAG_FASTCALL) && get_args_size( odp ) > 4) output( "\tpushl %%edx\n" ); output( "\tpushl %%ecx\n" ); output( "\tpushl %%eax\n" ); } @@ -966,7 +968,7 @@ void output_def_file( DLLSPEC *spec, int include_private ) else if (strcmp(name, odp->link_name)) /* try to reduce output */ { output( "=%s", odp->link_name ); - if (!kill_at && target_cpu == CPU_x86 && !(odp->flags & FLAG_THISCALL)) + if (!kill_at && target_cpu == CPU_x86 && !(odp->flags & (FLAG_THISCALL | FLAG_FASTCALL))) output( "@%d", at_param ); } break; diff --git a/tools/winebuild/winebuild.man.in b/tools/winebuild/winebuild.man.in index 4455e63..8f25052 100644 --- a/tools/winebuild/winebuild.man.in +++ b/tools/winebuild/winebuild.man.in @@ -304,6 +304,12 @@ The function uses the .I thiscall calling convention (first parameter in %ecx register on i386). .TP +.B -fastcall +The function uses the +.I fastcall +calling convention (first two parameters in %ecx/%edx registers on +i386). +.TP .RE .BI -arch= cpu\fR[\fB,\fIcpu\fR] The entry point is only available on the specified CPU

1 0

Alexandre Julliard : ntdll: Support relay debugging of fastcall entry points.
by Alexandre Julliard 28 Feb '19

28 Feb '19

Module: wine Branch: master Commit: 9c9efd706821b3e4ff3a4e3a473e1950d1fd4819 URL: https://source.winehq.org/git/wine.git/?a=commit;h=9c9efd706821b3e4ff3a4e3a… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Feb 27 17:12:15 2019 +0100 ntdll: Support relay debugging of fastcall entry points. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ntdll/relay.c | 14 ++++++++++---- 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/dlls/ntdll/relay.c b/dlls/ntdll/relay.c index 4ebd1fd..a205086 100644 --- a/dlls/ntdll/relay.c +++ b/dlls/ntdll/relay.c @@ -371,7 +371,11 @@ DECLSPEC_HIDDEN void * WINAPI relay_trace_entry( struct relay_descr *descr, unsi if (!is_ret_val( arg_types[i+1] )) TRACE( "," ); } *nb_args = pos; - if (arg_types[0] == 't') *nb_args |= 0x80000000; /* thiscall */ + if (arg_types[0] == 't') + { + *nb_args |= 0x80000000; /* thiscall/fastcall */ + if (arg_types[1] == 't') *nb_args |= 0x40000000; /* fastcall */ + } TRACE( ") ret=%08x\n", stack[-1] ); return entry_point->orig_func; } @@ -416,9 +420,8 @@ __ASM_GLOBAL_FUNC( relay_call, "pushl 8(%ebp)\n\t" "call " __ASM_NAME("relay_trace_entry") "\n\t" /* copy the arguments*/ - "movl -16(%ebp),%ecx\n\t" /* number of args */ + "movzwl -16(%ebp),%ecx\n\t" /* number of args */ "jecxz 1f\n\t" - "andl $0x7fffffff,%ecx\n\t" "leal 0(,%ecx,4),%edx\n\t" "subl %edx,%esp\n\t" "andl $~15,%esp\n\t" @@ -427,7 +430,10 @@ __ASM_GLOBAL_FUNC( relay_call, "rep; movsl\n\t" "testl $0x80000000,-16(%ebp)\n\t" /* thiscall */ "jz 1f\n\t" - "popl %ecx\n" + "popl %ecx\n\t" + "testl $0x40000000,-16(%ebp)\n\t" /* fastcall */ + "jz 1f\n\t" + "popl %edx\n" /* call the entry point */ "1:\tcall *%eax\n\t" "movl %eax,%esi\n\t"

1 0

Alexandre Julliard : ntdll: Use the fastcall flag for RtlInterlockedPushListSList.
by Alexandre Julliard 28 Feb '19

28 Feb '19

Module: wine Branch: master Commit: 24721eccac8c0f7d4a64daa5e6e4016ae22be61e URL: https://source.winehq.org/git/wine.git/?a=commit;h=24721eccac8c0f7d4a64daa5… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Feb 27 17:13:18 2019 +0100 ntdll: Use the fastcall flag for RtlInterlockedPushListSList. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- .../api-ms-win-core-interlocked-l1-1-0.spec | 2 +- dlls/kernel32/kernel32.spec | 2 +- dlls/kernelbase/kernelbase.spec | 2 +- dlls/ntdll/ntdll.spec | 2 +- dlls/ntdll/rtl.c | 15 ++++++--------- 5 files changed, 10 insertions(+), 13 deletions(-) diff --git a/dlls/api-ms-win-core-interlocked-l1-1-0/api-ms-win-core-interlocked-l1-1-0.spec b/dlls/api-ms-win-core-interlocked-l1-1-0/api-ms-win-core-interlocked-l1-1-0.spec index 8b2d3ac..0058b45 100644 --- a/dlls/api-ms-win-core-interlocked-l1-1-0/api-ms-win-core-interlocked-l1-1-0.spec +++ b/dlls/api-ms-win-core-interlocked-l1-1-0/api-ms-win-core-interlocked-l1-1-0.spec @@ -8,5 +8,5 @@ @ stdcall -arch=i386 InterlockedIncrement(ptr) kernel32.InterlockedIncrement @ stdcall InterlockedPopEntrySList(ptr) kernel32.InterlockedPopEntrySList @ stdcall InterlockedPushEntrySList(ptr ptr) kernel32.InterlockedPushEntrySList -@ stdcall -norelay InterlockedPushListSList(ptr ptr ptr long) kernel32.InterlockedPushListSList +@ stdcall -fastcall InterlockedPushListSList(ptr ptr ptr long) kernel32.InterlockedPushListSList @ stdcall QueryDepthSList(ptr) kernel32.QueryDepthSList diff --git a/dlls/kernel32/kernel32.spec b/dlls/kernel32/kernel32.spec index d925431..c8d52ce 100644 --- a/dlls/kernel32/kernel32.spec +++ b/dlls/kernel32/kernel32.spec @@ -961,7 +961,7 @@ @ stdcall -arch=i386 InterlockedIncrement(ptr) @ stdcall InterlockedPopEntrySList(ptr) ntdll.RtlInterlockedPopEntrySList @ stdcall InterlockedPushEntrySList(ptr ptr) ntdll.RtlInterlockedPushEntrySList -@ stdcall -norelay InterlockedPushListSList(ptr ptr ptr long) ntdll.RtlInterlockedPushListSList +@ stdcall -fastcall InterlockedPushListSList(ptr ptr ptr long) ntdll.RtlInterlockedPushListSList @ stdcall InterlockedPushListSListEx(ptr ptr ptr long) ntdll.RtlInterlockedPushListSListEx @ stub InvalidateConsoleDIBits @ stdcall InvalidateNLSCache() diff --git a/dlls/kernelbase/kernelbase.spec b/dlls/kernelbase/kernelbase.spec index 0b19212..9a014b2 100644 --- a/dlls/kernelbase/kernelbase.spec +++ b/dlls/kernelbase/kernelbase.spec @@ -821,7 +821,7 @@ @ stdcall -arch=i386 InterlockedIncrement(ptr) kernel32.InterlockedIncrement @ stdcall InterlockedPopEntrySList(ptr) kernel32.InterlockedPopEntrySList @ stdcall InterlockedPushEntrySList(ptr ptr) kernel32.InterlockedPushEntrySList -@ stdcall -norelay InterlockedPushListSList(ptr ptr ptr long) kernel32.InterlockedPushListSList +@ stdcall -fastcall InterlockedPushListSList(ptr ptr ptr long) kernel32.InterlockedPushListSList @ stdcall InterlockedPushListSListEx(ptr ptr ptr long) kernel32.InterlockedPushListSListEx @ stub InternalLcidToName @ stub Internal_EnumCalendarInfo diff --git a/dlls/ntdll/ntdll.spec b/dlls/ntdll/ntdll.spec index 649774f..6062cfb 100644 --- a/dlls/ntdll/ntdll.spec +++ b/dlls/ntdll/ntdll.spec @@ -730,7 +730,7 @@ @ stdcall RtlInterlockedFlushSList(ptr) @ stdcall RtlInterlockedPopEntrySList(ptr) @ stdcall RtlInterlockedPushEntrySList(ptr ptr) -@ stdcall -norelay RtlInterlockedPushListSList(ptr ptr ptr long) +@ stdcall -fastcall RtlInterlockedPushListSList(ptr ptr ptr long) @ stdcall RtlInterlockedPushListSListEx(ptr ptr ptr long) # @ stub RtlInvertRangeList @ stdcall RtlIpv4AddressToStringA(ptr ptr) diff --git a/dlls/ntdll/rtl.c b/dlls/ntdll/rtl.c index 393bd27..d13ec31 100644 --- a/dlls/ntdll/rtl.c +++ b/dlls/ntdll/rtl.c @@ -51,13 +51,15 @@ WINE_DEFAULT_DEBUG_CHANNEL(ntdll); #ifdef __i386__ -#define DEFINE_FASTCALL4_ENTRYPOINT( name ) \ - __ASM_STDCALL_FUNC( name, 16, \ +#define DEFINE_FASTCALL_WRAPPER(func,args) \ + __ASM_GLOBAL_FUNC( __fastcall_ ## func, \ "popl %eax\n\t" \ "pushl %edx\n\t" \ "pushl %ecx\n\t" \ "pushl %eax\n\t" \ - "jmp " __ASM_NAME("__regs_") #name __ASM_STDCALL(16)) + "jmp " __ASM_NAME(#func) __ASM_STDCALL(args) ) +#else +#define DEFINE_FASTCALL_WRAPPER(func,args) /* nothing */ #endif /* CRC polynomial 0xedb88320 */ @@ -1256,14 +1258,9 @@ PSLIST_ENTRY WINAPI RtlInterlockedPushListSListEx(PSLIST_HEADER list, PSLIST_ENT /************************************************************************* * RtlInterlockedPushListSList [NTDLL.@] */ -#ifdef DEFINE_FASTCALL4_ENTRYPOINT -DEFINE_FASTCALL4_ENTRYPOINT(RtlInterlockedPushListSList) -PSLIST_ENTRY WINAPI DECLSPEC_HIDDEN __regs_RtlInterlockedPushListSList(PSLIST_HEADER list, PSLIST_ENTRY first, - PSLIST_ENTRY last, ULONG count) -#else +DEFINE_FASTCALL_WRAPPER(RtlInterlockedPushListSList, 16) PSLIST_ENTRY WINAPI RtlInterlockedPushListSList(PSLIST_HEADER list, PSLIST_ENTRY first, PSLIST_ENTRY last, ULONG count) -#endif { return RtlInterlockedPushListSListEx(list, first, last, count); }

1 0