Re: [Mingw-w64-public] Building mingw-w64 and include paths

Let's try this again. This time the proposed patch is attached which may help. For 'ease of review,' this patch does not include the 'generated' makefile.in files. The problem I am trying to fix is that when building mingw-w64, the compiler will often use headers from the Tools Directories

Re: [Mingw-w64-public] [PATCH] Change check for _WIN64 to check for __x86_64__ in assembler

Hi Kai, On Aug 22 17:34, Kai Tietz wrote: > Hi Corinna, > > 2016-08-22 17:23 GMT+02:00 Corinna Vinschen : > > Lots of assembler files in the math subdir check for _WIN64 when they > > actually mean to check for the x86_64 target CPU, rather than the target > > OS. This

Re: [Mingw-w64-public] [PATCH] Align thread entry point stack

Hello Aleksey, 2016-08-22 13:52 GMT+02:00 Aleksey Vasenev : > __attribute__((aligned)) don't work for stack variables in threads created > with _beginthreadex without alignment. > > Signed-off-by: Aleksey Vasenev > --- > send it again > github:

Re: [Mingw-w64-public] GDB release canditate 7.11.90 build failure for mingw-w64

2016-08-22 17:40 GMT+02:00 Pierre Free Pascal : > > >> -Message d'origine- >> De : Kai Tietz [mailto:ktiet...@googlemail.com] >> Envoyé : lundi 22 août 2016 16:57 >> À : mingw-w64-public@lists.sourceforge.net >> Objet : Re: [Mingw-w64-public] GDB release canditate

[Mingw-w64-public] [PATCH] Align thread entry point stack

__attribute__((aligned)) don't work for stack variables in threads created with _beginthreadex without alignment. Signed-off-by: Aleksey Vasenev --- send it again github: https://github.com/Ratio2/mingw-w64/tree/align fix crash: ffmpeg -f lavfi -i testsrc -vcodec libvpx

Re: [Mingw-w64-public] [PATCH] Change check for _WIN64 to check for __x86_64__ in assembler

Hi Corinna, 2016-08-22 17:23 GMT+02:00 Corinna Vinschen : > Hi folks, > > > Lots of assembler files in the math subdir check for _WIN64 when they > actually mean to check for the x86_64 target CPU, rather than the target > OS. This patch fixes it. This helps Cygwin in the

[Mingw-w64-public] [PATCH] Change check for _WIN64 to check for __x86_64__ in assembler

Hi folks, Lots of assembler files in the math subdir check for _WIN64 when they actually mean to check for the x86_64 target CPU, rather than the target OS. This patch fixes it. This helps Cygwin in the first place because __x86_64__ is defined in both toolchains, while _WIN64 is only defined

[Mingw-w64-public] GDB release canditate 7.11.90 build failure for mingw-w64

Hi, this is my first email to this list. I would have directly put that in the ‘Tickets/Bugs’ of sourceforge, but I didn’t find where I should send a request to get my account accepted for bug reports. I am using GNU debugger GDB quite heavily, partly because I am the pascal

Re: [Mingw-w64-public] [PATCH] math: Preserve NAN/INF in rint(f) on ARM

Ok. Please apply. Important is here that we don't normalize NaN/Inf. I did so for pow, and well, it caused some troubles. Thanks, Kai 2016-08-22 15:41 GMT+02:00 Martin Storsjö : > rintl doesn't need changes since it calls rint(). > --- > mingw-w64-crt/math/rint.c | 2 ++ >

[Mingw-w64-public] [PATCH] math: Fix rounding in ceilf and floorf on ARM

If the 'r' flag is omitted from vcvt, the rounding mode specified in the fpscr is ignored. --- mingw-w64-crt/math/ceilf.S | 2 +- mingw-w64-crt/math/floorf.S | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/mingw-w64-crt/math/ceilf.S b/mingw-w64-crt/math/ceilf.S index

Re: [Mingw-w64-public] [PATCH] math: Fix rounding in ceilf and floorf on ARM

Thanks, please go ahead and apply. Regards, Kai 2016-08-22 15:42 GMT+02:00 Martin Storsjö : > If the 'r' flag is omitted from vcvt, the rounding mode specified > in the fpscr is ignored. > --- > mingw-w64-crt/math/ceilf.S | 2 +- > mingw-w64-crt/math/floorf.S | 2 +- > 2

Re: [Mingw-w64-public] [PATCH] math: Fix rounding in llrintl on ARM

Patch is ok. Please go ahead. Thanks, Kai 2016-08-22 15:40 GMT+02:00 Martin Storsjö : > --- > This is the same fix as in e9e42ce3113b which is already pushed; > I didn't notice that this function has a separate copy of the > implementation (contrary to e.g. rintl where the ARM

[Mingw-w64-public] [PATCH] math: Preserve NAN/INF in rint(f) on ARM

rintl doesn't need changes since it calls rint(). --- mingw-w64-crt/math/rint.c | 2 ++ mingw-w64-crt/math/rintf.c | 2 ++ 2 files changed, 4 insertions(+) diff --git a/mingw-w64-crt/math/rint.c b/mingw-w64-crt/math/rint.c index 01f9644..6d0b632 100644 --- a/mingw-w64-crt/math/rint.c +++

[Mingw-w64-public] [PATCH] math: Fix rounding in llrintl on ARM

--- This is the same fix as in e9e42ce3113b which is already pushed; I didn't notice that this function has a separate copy of the implementation (contrary to e.g. rintl where the ARM block just calls rint). --- mingw-w64-crt/math/llrintl.c | 11 ++- 1 file changed, 10 insertions(+), 1

Re: [Mingw-w64-public] _CRTIMP

Hmm, as all of those are functions, and we get those warnings during crt build. So we could simply take care that within crt build (and it helper libraries) those prototypes using a _CRTIMP variant without dllimport. This should work in all scenarios well, as linker will resolve such functions.

Re: [Mingw-w64-public] _CRTIMP

On 8/18/2016 11:27 PM, David Wohlferd wrote: > My next patch is very small, but it affects a bunch of code. Ponder > it a bit before approving. The goal is to fix all the warnings like > this: > > warning: '_unlock_file' redeclared without dllimport attribute: > previous dllimport ignored

Re: [Mingw-w64-public] undefined references to __GSHandlerCheck and __security_check_cookie

2016-08-22 8:58 GMT+02:00 Mario Emmenlauer : > > Dear All, > > I'm trying to link against MySQL, but it fails with undefined references > to __GSHandlerCheck and __security_check_cookie. I searched and this has > been asked before (several years back), but back then there was

Re: [Mingw-w64-public] [PATCH] variadic functions can't be stdcall

2016-08-22 9:11 GMT+02:00 Martin Storsjö : > On Sun, 21 Aug 2016, David Wohlferd wrote: > >> As my expert on patch etiquette, I have a question for you. When >> posting a patch, does one traditionally include all the files that will >> be in the push? Or do you skip the

Re: [Mingw-w64-public] [PATCH] for missing voids

2016-08-22 8:43 GMT+02:00 David Wohlferd : > On 8/21/2016 11:17 PM, Martin Storsjö wrote: >> On Sun, 21 Aug 2016, David Wohlferd wrote: >> >>> To my surprise, these two statements have (slightly) different meanings: >>> >>> STDAPI MFUnregisterPlatformFromMMCSS (); >>>

Re: [Mingw-w64-public] [PATCH] variadic functions can't be stdcall

On Sun, 21 Aug 2016, David Wohlferd wrote: > As my expert on patch etiquette, I have a question for you. When > posting a patch, does one traditionally include all the files that will > be in the push? Or do you skip the 'generated' files to make the review > easier? I guess it depends on the

[Mingw-w64-public] undefined references to __GSHandlerCheck and __security_check_cookie

Dear All, I'm trying to link against MySQL, but it fails with undefined references to __GSHandlerCheck and __security_check_cookie. I searched and this has been asked before (several years back), but back then there was no solution found. AFAIK these two where not available in MinGW's runtime,

Re: [Mingw-w64-public] [PATCH] variadic functions can't be stdcall

On 8/21/2016 11:19 PM, Martin Storsjö wrote: > On Sun, 21 Aug 2016, David Wohlferd wrote: > >> By definition, functions with variable numbers of parameters cannot be >> stdcall. Clang complains (warning: stdcall calling convention ignored on >> variadic function). >> >> Attached. > Seems ok to

Re: [Mingw-w64-public] [PATCH] for push/pop macro problem

On 8/21/2016 11:27 PM, Martin Storsjö wrote: > On Sun, 21 Aug 2016, David Wohlferd wrote: > >> Under certain circumstances, the #pragma pop_macro("__has_builtin") at the >> bottom of intrin-impl.h can be called without ever having hit the #pragma >> push_macro("__has_builtin") at the top. Clang

Re: [Mingw-w64-public] [PATCH] for missing voids

On 8/21/2016 11:17 PM, Martin Storsjö wrote: > On Sun, 21 Aug 2016, David Wohlferd wrote: > >> To my surprise, these two statements have (slightly) different meanings: >> >> STDAPI MFUnregisterPlatformFromMMCSS (); >> STDAPI MFUnregisterPlatformFromMMCSS (void); >> >> And clang complains about

Re: [Mingw-w64-public] [PATCH] for push/pop macro problem

On Sun, 21 Aug 2016, David Wohlferd wrote: > Under certain circumstances, the #pragma pop_macro("__has_builtin") at the > bottom of intrin-impl.h can be called without ever having hit the #pragma > push_macro("__has_builtin") at the top. Clang warns about this, so I have > moved the push

Re: [Mingw-w64-public] [PATCH] for missing voids

On Sun, 21 Aug 2016, David Wohlferd wrote: > To my surprise, these two statements have (slightly) different meanings: > > STDAPI MFUnregisterPlatformFromMMCSS (); > STDAPI MFUnregisterPlatformFromMMCSS (void); > > And clang complains about it (warning: function with no prototype cannot use >

Re: [Mingw-w64-public] [PATCH] for sinhl()

On Sun, 21 Aug 2016, David Wohlferd wrote: > In this function: > >long double sinhl(long double x) > > there is a call to fabs: > > (fabs (x) > (MAXLOGL + LOGE2L))) > > However, fabs doesn't take a (long double), it only takes a (double). Clang > complains about the truncation