Re: [Mingw-w64-public] [PATCH] headers: Add support for _FORTIFY_SOURCE=3

2023-02-08 Thread Christian Franke
LIU Hao wrote: 在 2023-02-08 20:50, Martin Storsjö 写道: Thanks, this patch seems to work well for me with Clang! If there are no other objections, I'd go ahead and push it. Mostly looks good, except:    +#ifndef __has_builtin    +  #define __has_builtin(x) 0    +#endif Do you have any prefer

Re: [Mingw-w64-public] [PATCH] headers: Add support for _FORTIFY_SOURCE=3

2023-02-08 Thread Martin Storsjö
On Wed, 8 Feb 2023, LIU Hao wrote: 在 2023-02-08 20:50, Martin Storsjö 写道: Thanks, this patch seems to work well for me with Clang! If there are no other objections, I'd go ahead and push it. Mostly looks good, except: +#ifndef __has_builtin + #define __has_builtin(x) 0 +#endif Do y

Re: [Mingw-w64-public] [PATCH] headers: Add support for _FORTIFY_SOURCE=3

2023-02-08 Thread LIU Hao
在 2023-02-08 20:50, Martin Storsjö 写道: Thanks, this patch seems to work well for me with Clang! If there are no other objections, I'd go ahead and push it. Mostly looks good, except: +#ifndef __has_builtin + #define __has_builtin(x) 0 +#endif Do you have any preferences on the ind

Re: [Mingw-w64-public] [PATCH] headers: Add support for _FORTIFY_SOURCE=3

2023-02-08 Thread Martin Storsjö
On Fri, 3 Feb 2023, Christian Franke wrote: LIU Hao wrote: 在 2023/2/2 06:41, Christian Franke 写道: This patch allows to use _FORTIFY_SOURCE=3 which is also supported by GLIBC headers. Thanks. This patch looks good to me. Pushed now. BTW does clang support this as well? I suspect we should