Re: [Mingw-w64-public] [PATCH] intrin-impl.h: Added __popcnt16, __popcnt, and __popcnt64.

2017-02-17 Thread Martell Malone
Hey guys, > Thanks, but I'd be more interested in documentation how to get cross > compilation setup with clang + mingw-w64. I did one in the past, but it > required patching clang and llvm and working around misc problems I > don't quite remember in details. > I haven't tried but I doubt it will

Re: [Mingw-w64-public] [PATCH] intrin-impl.h: Guard all intrins definitions by __has_builtin.

2017-02-17 Thread Kai Tietz
Patch is ok. Thanks, please apply. Kai 2017-02-17 13:25 GMT+01:00 Jacek Caban : > Please review. > > --- > mingw-w64-headers/include/psdk_inc/intrin-impl.h | 230 > ++- > 1 file changed, 226 insertions(+), 4 deletions(-) > > > > ---

Re: [Mingw-w64-public] [PATCH] intrin-impl.h: Guard all intrins definitions by __has_builtin.

2017-02-17 Thread David Grayson
If the compiler has a builtin for the intrinsic, why would mingw-w64 still provide a prototype for it? It seems like it would have no benefit and be likely to just cause compiler errors in the future. --David On Fri, Feb 17, 2017 at 4:25 AM, Jacek Caban wrote: > Please review. > > --- > mingw-

[Mingw-w64-public] [PATCH] intrin-impl.h: Guard all intrins definitions by __has_builtin.

2017-02-17 Thread Jacek Caban
Please review. --- mingw-w64-headers/include/psdk_inc/intrin-impl.h | 230 ++- 1 file changed, 226 insertions(+), 4 deletions(-) diff --git a/mingw-w64-headers/include/psdk_inc/intrin-impl.h b/mingw-w64-headers/include/psdk_inc/intrin-impl.h index fc781ff..c6b98f6 100644 ---

Re: [Mingw-w64-public] [PATCH] intrin-impl.h: Added __popcnt16, __popcnt, and __popcnt64.

2017-02-17 Thread Mateusz Mikuła
Hello Jacek, > Thanks for testing. Sadly, I found that the patch wasn't correct. Could you please try the attached one? New patch is also fine. > Thanks, but I'd be more interested in documentation how to get cross compilation setup with clang + mingw-w64. I did one in the past, but it required

Re: [Mingw-w64-public] [PATCH] intrin-impl.h: Added __popcnt16, __popcnt, and __popcnt64.

2017-02-17 Thread Jacek Caban
Hi Mateusz, On 17.02.2017 00:06, Mateusz Mikuła wrote: > Hello Jacek, > > With this patch clang builds example code from first message with and > without `-fms-extensions`. Thanks for testing. Sadly, I found that the patch wasn't correct. Could you please try the attached one? > I can provide pr

Re: [Mingw-w64-public] sqrt.patch from bug #567

2017-02-17 Thread JonY
On 02/17/2017 10:39 AM, Mateusz wrote: > And version 2 proposed by Kai Tietz. > > W dniu 2017-02-17 o 10:10, Mateusz pisze: >> Hi! >> >> There are lots of complaint about sqrt -- see bug #567. Martin Whitaker >> prepared a patch which I converted to git form for easy apply. >> >> Please review. >

Re: [Mingw-w64-public] sqrt.patch from bug #567

2017-02-17 Thread Mateusz
And version 2 proposed by Kai Tietz. W dniu 2017-02-17 o 10:10, Mateusz pisze: > Hi! > > There are lots of complaint about sqrt -- see bug #567. Martin Whitaker > prepared a patch which I converted to git form for easy apply. > > Please review. diff --git a/mingw-w64-crt/math/sqrt.def.h b/ming

[Mingw-w64-public] sqrt.patch from bug #567

2017-02-17 Thread Mateusz
Hi! There are lots of complaint about sqrt -- see bug #567. Martin Whitaker prepared a patch which I converted to git form for easy apply. Please review. diff --git a/mingw-w64-crt/math/sqrt.def.h b/mingw-w64-crt/math/sqrt.def.h index 602409a..b6f5460 100644 --- a/mingw-w64-crt/math/sqrt.def.h +