[PATCH 1/4] winevulkan: Define alloca and forceinline
Liam Middlebrook
lmiddlebrook at nvidia.com
Thu Mar 5 15:50:40 CST 2020
Joshua,
It looks like Marvin's getting a bit confused by your patches, here's what it's
trying (and failing) to apply:
https://testbot.winehq.org/GetFile.pl?JobKey=66437&StepKey=1
It looks like your email it tried to merge your plaintext email body with the
attachment, might be worth seeing what happens if you don't attach the .patch files
explicitly.a
The actual .patch file which you've attached applies correctly to my local tree.
On 3/5/20 9:03 AM, Joshua Ashton wrote:
> Sets up these definitions needed for future patches
>
> Signed-off-by: Joshua Ashton <joshua at froggi.es>
> ---
> dlls/winevulkan/vulkan_private.h | 12 ++++++++++++
> 1 file changed, 12 insertions(+)
>
> diff --git a/dlls/winevulkan/vulkan_private.h b/dlls/winevulkan/vulkan_private.h
> index 17072d2341..88233daced 100644
> --- a/dlls/winevulkan/vulkan_private.h
> +++ b/dlls/winevulkan/vulkan_private.h
> @@ -34,6 +34,18 @@
>
> #include "vulkan_thunks.h"
>
> +#if defined(__GNUC__) || defined(__clang__)
> +#include <alloca.h>
> +#define WINEVULKAN_FORCEINLINE __attribute__((always_inline)) inline
Rather than creating a new WINEVULKAN_FORCEINLINE definition here, I think you
should just be able to use the existing FORCEINLINE definition which is supplied by
winnt.h
Thanks,
Liam Middlebrook
> +#define WINEVULKAN_ALLOCA(x) alloca(x)
> +#elif defined(_MSC_VER)
> +#include <malloc.h>
> +#define WINEVULKAN_FORCEINLINE __forceinline
> +#define WINEVULKAN_ALLOCA(x) _alloca(x)
> +#else
> +#error Unsupported platform for forceinline/alloca
> +#endif
> +
> /* Magic value defined by Vulkan ICD / Loader spec */
> #define VULKAN_ICD_MAGIC_VALUE 0x01CDC0DE
>
>
-----------------------------------------------------------------------------------
This email message is for the sole use of the intended recipient(s) and may contain
confidential information. Any unauthorized review, use, disclosure or distribution
is prohibited. If you are not the intended recipient, please contact the sender by
reply email and destroy all copies of the original message.
-----------------------------------------------------------------------------------
More information about the wine-devel
mailing list