LGTM

On Sun, Nov 5, 2017 at 7:49 PM, Martin Storsjö <mar...@martin.st> wrote:

> Since clang 3.4, __debugbreak is limited to -fms-extensions
> and thus not available by default when targeting mingw.
>
> Signed-off-by: Martin Storsjö <mar...@martin.st>
> ---
>  mingw-w64-headers/crt/_mingw.h.in | 7 ++++++-
>  1 file changed, 6 insertions(+), 1 deletion(-)
>
> diff --git a/mingw-w64-headers/crt/_mingw.h.in b/mingw-w64-headers/crt/_
> mingw.h.in
> index c5f3bf8..c7b07ed 100644
> --- a/mingw-w64-headers/crt/_mingw.h.in
> +++ b/mingw-w64-headers/crt/_mingw.h.in
> @@ -534,8 +534,13 @@ typedef int __int128 __attribute__ ((__mode__ (TI)));
>  extern "C" {
>  #endif
>
> +
> +#ifndef __has_builtin
> +  #define __has_builtin(x) 0
> +#endif
> +
>  #ifdef __MINGW_INTRIN_INLINE
> -#if !defined (__clang__)
> +#if !defined (__clang__) || !__has_builtin(__debugbreak)
>  void __cdecl __debugbreak(void);
>  __MINGW_INTRIN_INLINE void __cdecl __debugbreak(void)
>  {
> --
> 2.7.4
>
>
> ------------------------------------------------------------
> ------------------
> Check out the vibrant tech community on one of the world's most
> engaging tech sites, Slashdot.org! http://sdm.link/slashdot
> _______________________________________________
> Mingw-w64-public mailing list
> Mingw-w64-public@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/mingw-w64-public
>
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Mingw-w64-public mailing list
Mingw-w64-public@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mingw-w64-public

Reply via email to