Re: [Mingw-w64-public] Clean build of mingw-w64 (ie no more warning)

2016-08-14 Thread Martin Storsjö
On Sat, 13 Aug 2016, dw wrote: > I still have some more fixes for ARM, but this patch is getting too big. > This is a logical point to break. Yes, that's probably for the best. If/when committing (iirc someone had already ok'd it?), I think it'd be even better to split it further, to one

Re: [Mingw-w64-public] Clean build of mingw-w64 (ie no more warning)

2016-08-14 Thread Martin Storsjö
On Sun, 14 Aug 2016, NightStrike wrote: On Sun, Aug 14, 2016 at 9:02 AM, Martin Storsjö <mar...@martin.st> wrote: On Sat, 13 Aug 2016, dw wrote: I still have some more fixes for ARM, but this patch is getting too big. This is a logical point to break. Yes, that's probably for th

Re: [Mingw-w64-public] Clean build of mingw-w64 (ie no more warning)

2016-08-15 Thread Martin Storsjö
On Sun, 14 Aug 2016, dw wrote: >>> - _vswprintf_p.c, _vscwprintf_p.c - these also add a comment that wasn't >>> there before. Probably ok, but I guess it's preferrable to have such >>> changes split out. >>> - aviriff.h, I see no other changes than adding in leading zeros > > Yes, that's

[Mingw-w64-public] [PATCH] stdio: Convert from 64 bit doubles to 80 bit, on platforms that lack an 80 bit long double

2016-08-09 Thread Martin Storsjö
This fixes printf of floats/doubles with -D__USE_MINGW_ANSI_STDIO=1, on arm. --- Using __fpclassify instead of __fpclassifyl for the 64 bit long double case, as suggested by Kai. --- mingw-w64-crt/stdio/mingw_pformat.c | 25 - 1 file changed, 24 insertions(+), 1

Re: [Mingw-w64-public] [PATCH 3/5] stdio: Convert from 64 bit doubles to 80 bit, on platforms that lack an 80 bit long double

2016-08-09 Thread Martin Storsjö
On Tue, 9 Aug 2016, Kai Tietz wrote: Hi, thank you for working on that. I have just a few nits. 2016-08-08 15:24 GMT+02:00 Martin Storsjö <mar...@martin.st>: This fixes printf of floats/doubles with -D__USE_MINGW_ANSI_STDIO=1, on arm. --- mingw-w64-crt/stdio/mingw_pformat.

[Mingw-w64-public] [PATCH 5/5] RFC: arm: Avoid making __fsqrt_internal a global symbol

2016-08-08 Thread Martin Storsjö
This fixes duplicate symbols if a calling app links in both e.g. sqrt and sqrtf. --- AFAIK, the .def is only there for debugging info - removing the .global doesn't seem to be enough to avoid the linker failures at least, but perhaps it can be made debug info for a static symbol in some other way?

[Mingw-w64-public] [PATCH 1/5] arm: Add missing it instructions

2016-08-08 Thread Martin Storsjö
When building in thumb mode, the conditional instructions are supposed to have 'it' instructions preceding them (unless the assembler is set to automatically inject such). This fixes building with clang/llvm 3.9, while the build passed with clang/llvm 3.8 (unsure whether it actually injected such

Re: [Mingw-w64-public] Clean build of mingw-w64 (ie no more warning)

2016-08-08 Thread Martin Storsjö
On Mon, 8 Aug 2016, dw wrote: > Q3: ARM is giving a bunch of warnings in the form: "dlltool.exe: Syntax error > in def file .../mingw-w64-crt/libarm32/powerwmiprovider.def:62." I have > looked at the def file, but I don't understand what I am seeing. How do I > diagnose/repair this? FWIW, I

[Mingw-w64-public] [PATCH 3/5] stdio: Convert from 64 bit doubles to 80 bit, on platforms that lack an 80 bit long double

2016-08-08 Thread Martin Storsjö
This fixes printf of floats/doubles with -D__USE_MINGW_ANSI_STDIO=1, on arm. --- mingw-w64-crt/stdio/mingw_pformat.c | 21 - 1 file changed, 20 insertions(+), 1 deletion(-) diff --git a/mingw-w64-crt/stdio/mingw_pformat.c b/mingw-w64-crt/stdio/mingw_pformat.c index

Re: [Mingw-w64-public] [PATCH 4/5] RFC: arm: Make sure to at least run a certain number of rounds in 'log' functions

2016-08-08 Thread Martin Storsjö
On Mon, 8 Aug 2016, Martin Storsjö wrote: --- The value 30 is a blind guesstimate of what's sensible; with that, I get log() returning values close to the real values (differing only in the third digit or so). I'm not sure exactly how the original logic for the number of rounds is supposed

Re: [Mingw-w64-public] Clean build of mingw-w64 (ie no more warning)

2016-08-16 Thread Martin Storsjö
On Mon, 15 Aug 2016, dw wrote: On 8/15/2016 4:54 AM, Martin Storsjö wrote: but 30 of these would be a huge hassle for the approvers as well. Not really, if they are well contained with an explanation matching it close by, it's trivial to review. Historically my experiences haven't been good

Re: [Mingw-w64-public] Clean build of mingw-w64 (ie no more warning)

2016-08-16 Thread Martin Storsjö
On Tue, 16 Aug 2016, dw wrote: On 8/15/2016 11:45 PM, Martin Storsjö wrote: Remove the extra copyright disclaimer headers that you're adding. If you're keen on adding them, send a separate patch for that. I have removed the 'extra' text (attached). Note that I am attaching 2 files

Re: [Mingw-w64-public] Clean build of mingw-w64 (ie no more warning)

2016-08-16 Thread Martin Storsjö
On Tue, 16 Aug 2016, dw wrote: > Attempting to follow Martin's suggestions, I'm attaching the next three (I > *think* this is the organization he requested). Ok to push? uchar.patch and ntsecapi.patch are ok with me. > = defines.patch > Fix minor variations in definitions

Re: [Mingw-w64-public] Clean build of mingw-w64 (ie no more warning)

2016-08-16 Thread Martin Storsjö
On Tue, 16 Aug 2016, David Wohlferd wrote: On 8/16/2016 7:30 AM, Martin Storsjö wrote: Btw, All of your mails end up sorted in the spam box by gmail (at least for me), citing this reason: "It has a from address in yahoo.com but has failed yahoo.com's required tests for authentic

[Mingw-w64-public] [PATCH] math: Fix rounding in llrint(f) on ARM

2016-08-18 Thread Martin Storsjö
--- Updated to check with fegetround to see which rounding mode to use. --- mingw-w64-crt/math/llrint.c | 11 ++- mingw-w64-crt/math/llrintf.c | 11 ++- 2 files changed, 20 insertions(+), 2 deletions(-) diff --git a/mingw-w64-crt/math/llrint.c b/mingw-w64-crt/math/llrint.c index

Re: [Mingw-w64-public] [PATCH] math: Fix rounding in llrint(f) on ARM

2016-08-18 Thread Martin Storsjö
On Thu, 18 Aug 2016, Kai Tietz wrote: Hello Martin, 2016-08-18 15:31 GMT+02:00 Martin Storsjö <mar...@martin.st>: --- The (l)rint(f) functions call an inline assembly snippet to do the rounding - that does seem to round in the right way, but I'm not sure if that relies on the fpscr bei

Re: [Mingw-w64-public] [PATCH] math: Fix rounding in llrint(f) on ARM

2016-08-18 Thread Martin Storsjö
On Thu, 18 Aug 2016, Martin Storsjö wrote: On Thu, 18 Aug 2016, Kai Tietz wrote: Hello Martin, 2016-08-18 15:31 GMT+02:00 Martin Storsjö <mar...@martin.st>: --- The (l)rint(f) functions call an inline assembly snippet to do the rounding - that does seem to round in the right way, b

[Mingw-w64-public] [PATCH] math: Fix rounding in llrint(f) on ARM

2016-08-18 Thread Martin Storsjö
--- Updated to check with fegetround to see which rounding mode to use, fixed the way the mode is checked (by using == instead of &). --- mingw-w64-crt/math/llrint.c | 11 ++- mingw-w64-crt/math/llrintf.c | 11 ++- 2 files changed, 20 insertions(+), 2 deletions(-) diff --git

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

2016-08-22 Thread Martin Storsjö
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

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

2016-08-22 Thread Martin Storsjö
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 push/pop macro problem

2016-08-22 Thread Martin Storsjö
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] variadic functions can't be stdcall

2016-08-22 Thread 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 'generated' files to make the review > easier? I guess it depends on the

Re: [Mingw-w64-public] Clean build of mingw-w64 (ie no more warning)

2016-08-16 Thread Martin Storsjö
On Tue, 16 Aug 2016, dw wrote: > On 8/16/2016 4:16 AM, Jacek Caban wrote: >> On 16.08.2016 12:50, dw wrote: >>> --- a/mingw-w64-headers/include/mfidl.h >>> +++ b/mingw-w64-headers/include/mfidl.h >> This file is auto generated. You should not touch it, please change .idl >> file instead. > >

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

2016-08-22 Thread 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 block just calls rint). --- mingw-w64-crt/math/llrintl.c | 11 ++- 1 file changed, 10 insertions(+), 1

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

2016-08-22 Thread Martin Storsjö
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 ceilf and floorf on ARM

2016-08-22 Thread 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 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 llrint(f) on ARM

2016-08-18 Thread Martin Storsjö
On Thu, 18 Aug 2016, Martin Storsjö wrote: --- Updated to check with fegetround to see which rounding mode to use. --- mingw-w64-crt/math/llrint.c | 11 ++- mingw-w64-crt/math/llrintf.c | 11 ++- 2 files changed, 20 insertions(+), 2 deletions(-) diff --git a/mingw-w64-crt/math

[Mingw-w64-public] CRT math implementations on ARM

2016-09-07 Thread Martin Storsjö
Hi, In my testing with mingw-w64 on ARM, I've run into a number of issues with the implementation of the math routines. Some of these issues are: The whole family of log functions is problematic (even after my fix for the number of iterations in 41de4baaccba), e.g. log2() only returns values

Re: [Mingw-w64-public] Clean build of mingw-w64 (ie no more warning)

2016-08-18 Thread Martin Storsjö
On Thu, 18 Aug 2016, David Wohlferd wrote: > >> I assume I still need to wait for someone else to approve this before I >> push? > > Kai signed off on these via IRC. > > It's late, so just 3 quick new ones tonight: > > e_pow.patch - Signed/unsigned compare > > mingw_pformat.patch - Don't use

[Mingw-w64-public] [PATCH] math: Make sure NAN/INF is preserved in floor/ceil functions on ARM

2016-08-18 Thread Martin Storsjö
--- The indentation probably looks wonky in ths patch, but I'm trying to match the style of the rest of the arm assembly below, which uses tabs, equal to 4 spaces. This probably also should be done to a lot of other math functions, but I only ran into this as an issue with floor, in the libav

[Mingw-w64-public] [PATCH] math: Fix rounding in llrint(f) on ARM

2016-08-18 Thread Martin Storsjö
--- The (l)rint(f) functions call an inline assembly snippet to do the rounding - that does seem to round in the right way, but I'm not sure if that relies on the fpscr being set in the right mode? --- mingw-w64-crt/math/llrint.c | 2 +- mingw-w64-crt/math/llrintf.c | 2 +- 2 files changed, 2

Re: [Mingw-w64-public] [PATCH] math: Make sure NAN/INF is preserved in floor/ceil functions on ARM

2016-08-23 Thread Martin Storsjö
On Thu, 18 Aug 2016, Martin Storsjö wrote: --- The indentation probably looks wonky in ths patch, but I'm trying to match the style of the rest of the arm assembly below, which uses tabs, equal to 4 spaces. This probably also should be done to a lot of other math functions, but I only ran

Re: [Mingw-w64-public] CRT math implementations on ARM

2016-09-28 Thread Martin Storsjö
On Wed, 7 Sep 2016, Martin Storsjö wrote: Hi, In my testing with mingw-w64 on ARM, I've run into a number of issues with the implementation of the math routines. Some of these issues are: The whole family of log functions is problematic (even after my fix for the number of iterations

[Mingw-w64-public] [PATCH] Remove ARM softfloat math function implementations

2016-09-28 Thread Martin Storsjö
Just pass calls to msvcrt instead. The msvcrt math functions might not be fully C99 compliant, but the current softfloat implementations are lacking in many other ways (e.g. crashing due to infinite recursion for some input values, only working for certain ranges of input values, etc.). For

Re: [Mingw-w64-public] [PATCH] Remove ARM softfloat math function implementations

2016-09-28 Thread Martin Storsjö
On Wed, 28 Sep 2016, Martin Storsjö wrote: Just pass calls to msvcrt instead. The msvcrt math functions might not be fully C99 compliant, but the current softfloat implementations are lacking in many other ways (e.g. crashing due to infinite recursion for some input values, only working

[Mingw-w64-public] [PATCH] Make ARM softmath functions experimental, disabled by default

2016-10-03 Thread Martin Storsjö
By default, just forward calls to msvcrt.dll for the functions implemented with softmath routines. The msvcrt math functions might not be fully C99 compliant, but the current softfloat implementations are lacking in many other ways (e.g. crashing due to infinite recursion for some input values,

[Mingw-w64-public] [PATCH] intrin-impl.h: Fix mismatched ifdefs

2017-04-03 Thread Martin Storsjö
072254d added mismatched ifdefs for _InterlockedCompareExchange64 (only added an #if with no matching #endif) and for __readfsdword (only added a new #endif to an #if that already had a matching #endif). This fixes compilation for arm. --- mingw-w64-headers/include/psdk_inc/intrin-impl.h | 2 +-

Re: [Mingw-w64-public] [PATCH] Handle __CTOR_LIST__ for clang

2017-08-02 Thread Martin Storsjö
Hi Martell, André and Kai, On Sun, 28 Aug 2016, André Hentschel wrote: Am 09.08.2016 um 11:16 schrieb Kai Tietz: Hallo Martell, patch is ok. Wouldn't it be better to have those symbols in linker scrpt instead? That is actually the way used in ld for it. Thanlks, Kai 2016-08-06 5:14

Re: [Mingw-w64-public] [PATCH] Add a dlltool machine flag for targeting arm

2017-08-02 Thread Martin Storsjö
On Wed, 2 Aug 2017, JonY via Mingw-w64-public wrote: On 07/30/2017 07:46 PM, Martin Storsjö wrote: The GNU binutils dlltool doesn't actually support this target, but llvm-dlltool does. --- mingw-w64-crt/Makefile.am | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/mingw-w64

[Mingw-w64-public] [PATCH] Add function aliases in libmsvcrt.a for arm just like on x86/x86_64

2017-08-03 Thread Martin Storsjö
Function aliases aren't supported in genlib though, only in GNU binutils dlltool (which doesn't support arm) and llvm-dlltool. It didn't make sense to do this until now, when we have a dlltool that supports arm. This adds the function aliases from msvcrt-common.def.in, functions without a leading

Re: [Mingw-w64-public] [PATCH] Add the sincos* functions on arm, calling into sin/cos separately

2017-08-03 Thread Martin Storsjö
net>: On 07/30/2017 07:45 PM, Martin Storsjö wrote: On arm (unless --enable-experimental=softmath is set), we pass sin/cos function calls straight through to msvcrt. We normally provide implementations of the sincos family of functions, that provide both return values at once. Provide implement

[Mingw-w64-public] New compiler warnings (unused variable)

2017-08-03 Thread Martin Storsjö
Hi niXman, The recent commit you pushed, "_mingw_no_trailing_slash() for _stat64i32() and _wstat64i32()", added compiler warnings about an unused variable "int start", in case you want to fix it yourself. // Martin

[Mingw-w64-public] [PATCH 2/5] math: Avoid defining full functions with inline assembly

2017-08-03 Thread Martin Storsjö
Whatever compiler issue this tried to work around, it seems like it works with modern clang (which is the only supported compiler for arm in mingw at the moment) in any case. --- mingw-w64-crt/math/lrint.c| 19 +-- mingw-w64-crt/math/lrintf.c | 18 --

[Mingw-w64-public] [PATCH 1/5] math: Hook up the return value for sqrt on arm

2017-08-03 Thread Martin Storsjö
--- mingw-w64-crt/math/sqrt.def.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/mingw-w64-crt/math/sqrt.def.h b/mingw-w64-crt/math/sqrt.def.h index 0cd401d..adb7cf8 100644 --- a/mingw-w64-crt/math/sqrt.def.h +++ b/mingw-w64-crt/math/sqrt.def.h @@ -88,7 +88,7 @@ __FLT_ABI

[Mingw-w64-public] [PATCH 4/5] stdio: Pass the function to call to the v*scanf asm wrapper

2017-08-03 Thread Martin Storsjö
This unifies the wrappers, allowing to share them as on x86. --- mingw-w64-crt/stdio/vfscanf.c | 14 ++ mingw-w64-crt/stdio/vfwscanf.c | 14 ++ mingw-w64-crt/stdio/vsscanf.c | 14 ++ mingw-w64-crt/stdio/vswscanf.c | 14 ++ 4 files changed, 24

[Mingw-w64-public] [PATCH 5/5] stdio: Share the v*scanf assembly wrapper in the scanf.S external asm file

2017-08-03 Thread Martin Storsjö
This reduces code duplication. --- mingw-w64-crt/stdio/scanf.S| 30 +- mingw-w64-crt/stdio/vfscanf.c | 35 --- mingw-w64-crt/stdio/vfwscanf.c | 35 --- mingw-w64-crt/stdio/vsscanf.c | 35

[Mingw-w64-public] [PATCH 3/5] stdio: Simplify the arm assembly wrapper function for v*scanf

2017-08-03 Thread Martin Storsjö
Don't store below the stack pointer, but decrement it before writing; use loads/stores with post increment to avoid manually incrementing the pointer afterwards. Use subs instead of sub+cmp. --- mingw-w64-crt/stdio/vfscanf.c | 18 ++ mingw-w64-crt/stdio/vfwscanf.c | 18

[Mingw-w64-public] [PATCH] intrin-impl.h: Fix __buildbittesti for arm

2017-08-04 Thread Martin Storsjö
The local variables contain the full 32 bit value read from the target, and need to be stored in 32 bit variables. The 'old' value read is the full value of the memory location, so we need to extract the specific bit. Mark the output registers as earlyclobber, to make sure they don't alias the

[Mingw-w64-public] [PATCH] stdio: Share the frontend call wrapper assembly across the v*scanf functions

2017-08-04 Thread Martin Storsjö
The code is already structured as a template, with the file specific definitions at the top. Share the common template content in an included file instead, reducing duplication. --- mingw-w64-crt/Makefile.am | 3 ++- mingw-w64-crt/stdio/scanf2-template.S | 30

[Mingw-w64-public] [PATCH] math: Add a few more missing return statements in arm ifdefs, for __isnanl and __signbitl

2017-08-04 Thread Martin Storsjö
--- mingw-w64-headers/crt/math.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/mingw-w64-headers/crt/math.h b/mingw-w64-headers/crt/math.h index 25452a5..6d82c9f 100644 --- a/mingw-w64-headers/crt/math.h +++ b/mingw-w64-headers/crt/math.h @@ -575,7 +575,7 @@

[Mingw-w64-public] [PATCH 12/12] crt: Share defs where libarm32 has got a superset of the functions in lib64

2017-08-15 Thread Martin Storsjö
In these cases, use the libarm32 version for both. None of the differences should be architecture specific. Signed-off-by: Martin Storsjö <mar...@martin.st> --- mingw-w64-crt/Makefile.am | 4 +- mingw-w64-crt/{libarm32 => lib-common}/atl.def | 0 ming

[Mingw-w64-public] [PATCH 02/12] crt: Remove the executable bit from def files

2017-08-15 Thread Martin Storsjö
Signed-off-by: Martin Storsjö <mar...@martin.st> --- mingw-w64-crt/lib-common/devrtl.def | 0 mingw-w64-crt/lib32/user32.def | 0 mingw-w64-crt/lib64/devobj.def | 0 3 files changed, 0 insertions(+), 0 deletions(-) mode change 100755 => 100644 mingw-w64-crt/lib-common/devrtl.

[Mingw-w64-public] [PATCH 06/12] crt: Remove even more def files that only contain DLL hooks or similar

2017-08-15 Thread Martin Storsjö
These defs only export DLL hooks, and one of the following entry points: - GetProxyDllInfo - GetResourceString - InitHelperDll - ServiceEntry - ServiceMain Signed-off-by: Martin Storsjö <mar...@martin.st> --- mingw-w64-crt/lib64/Makefile.am | 23 --- mingw-w

[Mingw-w64-public] [PATCH 01/12] crt: Convert def files from dos newline to unix newlines

2017-08-15 Thread Martin Storsjö
Signed-off-by: Martin Storsjö <mar...@martin.st> --- mingw-w64-crt/lib32/d3dcompiler_47.def | 72 +- mingw-w64-crt/lib64/d3dcompiler_47.def | 72 +- 2 files changed, 72 insertions(+), 72 deletions(-) diff --git a/mingw-w64-crt

[Mingw-w64-public] [PATCH 07/12] crt: Remove defs that only contain C++ functions

2017-08-15 Thread Martin Storsjö
Since the mingw C++ runtime isn't compatible with the MSVC C++ runtime (and uses a completely different name mangling), these def files aren't of much use. Signed-off-by: Martin Storsjö <mar...@martin.st> --- mingw-w64-crt/lib32/Makefile.am |1 - mingw-w64-crt

[Mingw-w64-public] [PATCH 00/12] Def file cleanups and merging

2017-08-15 Thread Martin Storsjö
to be done will follow later. Martin Storsjö (12): crt: Convert def files from dos newline to unix newlines crt: Remove the executable bit from def files crt: Remove def files for DLLs with code page translations crt: Remove a nonsense def for a modem driver DLL crt: Remove printer monitor DLL

[Mingw-w64-public] [PATCH 05/12] crt: Remove printer monitor DLL def files

2017-08-15 Thread Martin Storsjö
Signed-off-by: Martin Storsjö <mar...@martin.st> --- mingw-w64-crt/lib-common/inetpp.def | 9 - mingw-w64-crt/lib-common/inetppui.def | 9 - mingw-w64-crt/lib64/Makefile.am | 13 - mingw-w64-crt/lib64/cnbjmon.def | 9 - mingw-w64-crt

[Mingw-w64-public] [PATCH 09/12] crt: Share def files that only differ in unnamed ordinals between lib64 and libarm32

2017-08-15 Thread Martin Storsjö
Signed-off-by: Martin Storsjö <mar...@martin.st> --- mingw-w64-crt/Makefile.am | 6 +- mingw-w64-crt/{lib64 => lib-common}/comdlg32.def | 0 mingw-w64-crt/{lib64 => lib-common}/gdiplus.def | 0 mingw-w64-crt/{lib64 => lib-common}/rpcss.def | 0

[Mingw-w64-public] [PATCH 03/12] crt: Remove def files for DLLs with code page translations

2017-08-15 Thread Martin Storsjö
These aren't DLL files that one links to directly. Signed-off-by: Martin Storsjö <mar...@martin.st> --- mingw-w64-crt/lib-common/c_is2022.def | 9 - mingw-w64-crt/lib-common/c_iscii.def | 9 - mingw-w64-crt/lib64/Makefile.am | 3 --- mingw-w64-crt/lib64/c_g180

[Mingw-w64-public] [PATCH 10/12] crt: Share defs that only differ in dll entry points between lib64 and libarm32

2017-08-15 Thread Martin Storsjö
Signed-off-by: Martin Storsjö <mar...@martin.st> --- mingw-w64-crt/{lib64 => lib-common}/acledit.def | 0 mingw-w64-crt/{lib64 => lib-common}/activeds.def | 0 mingw-w64-crt/{lib64 => lib-common}/appmgr.def | 0 mingw-w64-crt/{lib64 => lib-common}/asycfilt.def | 0 mi

[Mingw-w64-public] [PATCH 08/12] crt: Share identical def files between lib64 and libarm32

2017-08-15 Thread Martin Storsjö
These were missed before due to the differing line endings. Signed-off-by: Martin Storsjö <mar...@martin.st> --- mingw-w64-crt/Makefile.am | 2 +- .../{lib64 => lib-common}/d3dcompiler_47.def | 0 mingw-w64-crt/libarm32/d3dcompiler_47.def

[Mingw-w64-public] [PATCH 11/12] crt: Share defs where lib64 has got a superset of the functions in libarm32

2017-08-15 Thread Martin Storsjö
In these cases, use the lib64 version for both. None of the differences should be architecture specific. Signed-off-by: Martin Storsjö <mar...@martin.st> --- mingw-w64-crt/{lib64 => lib-common}/dnsapi.def | 0 mingw-w64-crt/{lib64 => lib-common}/dsrole.def | 0 mingw-w6

[Mingw-w64-public] [PATCH 04/12] crt: Remove a nonsense def for a modem driver DLL

2017-08-15 Thread Martin Storsjö
Signed-off-by: Martin Storsjö <mar...@martin.st> --- mingw-w64-crt/lib64/Makefile.am | 1 - mingw-w64-crt/lib64/agrmco64.def | 9 - 2 files changed, 10 deletions(-) delete mode 100644 mingw-w64-crt/lib64/agrmco64.def diff --git a/mingw-w64-crt/lib64/Makefile.am b/mingw-w64-crt

Re: [Mingw-w64-public] [PATCH 1/3] crt: Remove def files that only contain a ServiceMain function

2017-08-10 Thread Martin Storsjö
On Thu, 10 Aug 2017, JonY via Mingw-w64-public wrote: On 08/10/2017 01:08 PM, Martin Storsjö wrote: --- All 3 are OK, I'd prefer if the patches are signed off and patches sent with a cover letter to describe the overall patch series. Sure - I'll sign-off before pushing, but I'll start

Re: [Mingw-w64-public] [PATCH 03/18] crt: Add an initial libarm64 directory with def files

2017-08-10 Thread Martin Storsjö
On Wed, 9 Aug 2017, Martell Malone wrote: Martin added a patch to add support for stripping the @4, @8, @12 for llvm-dlltool which led us to discover this issue. http://cygwin.com/ml/binutils/2004-09/msg00031.html https://msdn.microsoft.com/en-us/library/deaxefa7.aspx With that in mind we

Re: [Mingw-w64-public] [PATCH] Handle __CTOR_LIST__ for clang

2017-08-14 Thread Martin Storsjö
On Sat, 5 Aug 2017, David Grayson wrote: Oops, here is the patch. On Sat, Aug 5, 2017 at 10:14 AM, David Grayson wrote: I think Martell's last patch would have worked but it's not as safe as I would like it to be. I think the constructor and destructor lists should

Re: [Mingw-w64-public] [PATCH] Handle __CTOR_LIST__ for clang

2017-08-14 Thread Martin Storsjö
On Mon, 14 Aug 2017, Martell Malone wrote: Hey Martin, I had a discussion with Nick and CC'd kai on getting the binutils patch in tree a week ago. https://sourceware.org/ml/binutils/2017-08/msg00123.html It seems to be moving along. Oh, I see - ok. Can you briefly summarize what change this

[Mingw-w64-public] [PATCH 10/27] crt: Unify cryptnet.def for lib64 and libarm32

2017-08-17 Thread Martin Storsjö
Signed-off-by: Martin Storsjö <mar...@martin.st> --- .../{libarm32 => lib-common}/cryptnet.def | 7 ++ mingw-w64-crt/lib64/cryptnet.def | 27 -- 2 files changed, 2 insertions(+), 32 deletions(-) rename mingw-w64-crt/{libarm32 =>

[Mingw-w64-public] [PATCH 01/27] crt: Include the def file in libwindowscodecs.a

2017-08-17 Thread Martin Storsjö
Previously, we didn't actually ever include the def file in this library. Signed-off-by: Martin Storsjö <mar...@martin.st> --- mingw-w64-crt/Makefile.am | 3 +++ mingw-w64-crt/lib32/Makefile.am| 1 - mingw-w64-crt/libarm32/Makefile.am | 1 - 3 files changed, 3 insertions

[Mingw-w64-public] [PATCH 05/27] crt: Unify advpack.def for lib64 and libarm32

2017-08-17 Thread Martin Storsjö
Signed-off-by: Martin Storsjö <mar...@martin.st> --- mingw-w64-crt/{libarm32 => lib-common}/advpack.def | 10 +++--- mingw-w64-crt/lib64/advpack.def| 40 -- 2 files changed, 5 insertions(+), 45 deletions(-) rename mingw-w64-crt/{libarm32 =>

[Mingw-w64-public] [PATCH 03/27] crt: Share a def file where lib64 has got a superset of the functions in libarm32

2017-08-17 Thread Martin Storsjö
Include the C++ function definitions only on x86_64, since they were only present in the lib64 def file. Signed-off-by: Martin Storsjö <mar...@martin.st> --- mingw-w64-crt/Makefile.am | 6 ++-- .../clbcatq.def => lib-common/clbcatq.def.in}

[Mingw-w64-public] [PATCH 16/27] crt: Unify kernel32.def for lib64 and libarm32

2017-08-17 Thread Martin Storsjö
Keep functions from both versions that were missing on the other ones. Include *Ums* functions only for x86_64, based on annotations in kernel32.spec in wine. Keep manually commented out functions as they were. Include __chkstk on arm though. Signed-off-by: Martin Storsjö <mar...@martin

[Mingw-w64-public] [PATCH 09/27] crt: Unify cryptext.def for lib64 and libarm32

2017-08-17 Thread Martin Storsjö
Signed-off-by: Martin Storsjö <mar...@martin.st> --- .../{libarm32 => lib-common}/cryptext.def | 7 ++-- mingw-w64-crt/lib64/cryptext.def | 40 -- 2 files changed, 2 insertions(+), 45 deletions(-) rename mingw-w64-crt/{libarm32 =>

[Mingw-w64-public] [PATCH 11/27] crt: Unify cryptui.def for lib64 and libarm32

2017-08-17 Thread Martin Storsjö
Signed-off-by: Martin Storsjö <mar...@martin.st> --- mingw-w64-crt/{libarm32 => lib-common}/cryptui.def | 19 ++-- mingw-w64-crt/lib64/cryptui.def| 56 -- 2 files changed, 14 insertions(+), 61 deletions(-) rename mingw-w64-crt/{libarm32 =>

[Mingw-w64-public] [PATCH 22/27] crt: Unify shell32.def for lib64 and libarm32

2017-08-17 Thread Martin Storsjö
Keeping all function definitions from both previous versions, since there's no clear indication that they would be architecture specific. Signed-off-by: Martin Storsjö <mar...@martin.st> --- mingw-w64-crt/Makefile.am | 4 +- mingw-w64-crt/{lib64 => lib-common}/sh

[Mingw-w64-public] [PATCH 21/27] crt: Unify schannel.def for lib64 and libarm32

2017-08-17 Thread Martin Storsjö
Signed-off-by: Martin Storsjö <mar...@martin.st> --- mingw-w64-crt/{lib64 => lib-common}/schannel.def | 9 ++--- mingw-w64-crt/libarm32/schannel.def | 42 2 files changed, 3 insertions(+), 48 deletions(-) rename mingw-w64-crt/{lib64 =>

[Mingw-w64-public] [PATCH 07/27] crt: Unify comctl32.def for lib64 and libarm32

2017-08-17 Thread Martin Storsjö
Keep functions from both. Signed-off-by: Martin Storsjö <mar...@martin.st> --- mingw-w64-crt/{lib64 => lib-common}/comctl32.def | 17 ++- mingw-w64-crt/libarm32/comctl32.def | 185 --- 2 files changed, 11 insertions(+), 191 deletions(-) rename ming

[Mingw-w64-public] [PATCH 04/27] crt: Unify advapi32.def for lib64 and libarm32

2017-08-17 Thread Martin Storsjö
All functions seem to be architecture independent except one. Keep MD5Update marked as DATA, as it was in lib64 before. Signed-off-by: Martin Storsjö <mar...@martin.st> --- mingw-w64-crt/Makefile.am | 4 +- .../advapi32.def => lib-common/advapi32.def.in}

[Mingw-w64-public] [PATCH 12/27] crt: Unify dmutil.def for lib64 and libarm32

2017-08-17 Thread Martin Storsjö
Signed-off-by: Martin Storsjö <mar...@martin.st> --- mingw-w64-crt/{libarm32 => lib-common}/dmutil.def | 6 +--- mingw-w64-crt/lib64/dmutil.def| 37 --- 2 files changed, 1 insertion(+), 42 deletions(-) rename mingw-w64-crt/{libarm32 =>

[Mingw-w64-public] [PATCH 06/27] crt: Unify bcrypt.def for lib64 and libarm32

2017-08-17 Thread Martin Storsjö
Keep functions from both previous versions. Signed-off-by: Martin Storsjö <mar...@martin.st> --- mingw-w64-crt/{lib64 => lib-common}/bcrypt.def | 8 ++-- mingw-w64-crt/libarm32/bcrypt.def | 64 -- 2 files changed, 3 insertions(+), 69 deletions(-

[Mingw-w64-public] [PATCH 13/27] crt: Unify duser.def for lib64 and libarm32

2017-08-17 Thread Martin Storsjö
Signed-off-by: Martin Storsjö <mar...@martin.st> --- mingw-w64-crt/{libarm32 => lib-common}/duser.def | 12 ++- mingw-w64-crt/lib64/duser.def| 120 --- 2 files changed, 7 insertions(+), 125 deletions(-) rename mingw-w64-crt/{libarm32 =>

[Mingw-w64-public] [PATCH 02/27] crt: Add a framework for sharing def files with arch specific differences

2017-08-17 Thread Martin Storsjö
Signed-off-by: Martin Storsjö <mar...@martin.st> --- mingw-w64-crt/Makefile.am | 10 + mingw-w64-crt/def-include/func.def.in | 42 +++ 2 files changed, 52 insertions(+) create mode 100644 mingw-w64-crt/def-include/func.def.in diff

[Mingw-w64-public] [PATCH 08/27] crt: Unify crypt32.def for lib64 and libarm32

2017-08-17 Thread Martin Storsjö
Signed-off-by: Martin Storsjö <mar...@martin.st> --- mingw-w64-crt/{libarm32 => lib-common}/crypt32.def | 23 +- mingw-w64-crt/lib64/crypt32.def| 273 - 2 files changed, 6 insertions(+), 290 deletions(-) rename mingw-w64-crt/{libarm32 =>

[Mingw-w64-public] [PATCH 24/27] crt: Unify winmm.def for lib64 and libarm32

2017-08-17 Thread Martin Storsjö
Keeping all function definitions from both previous versions. Signed-off-by: Martin Storsjö <mar...@martin.st> --- mingw-w64-crt/{lib64 => lib-common}/winmm.def | 21 ++- mingw-w64-crt/libarm32/winmm.def | 200 -- 2 files changed, 14 insertion

[Mingw-w64-public] [PATCH 20/27] crt: Unify rpcrt4.def for lib64 and libarm32

2017-08-17 Thread Martin Storsjö
Signed-off-by: Martin Storsjö <mar...@martin.st> --- mingw-w64-crt/{lib64 => lib-common}/rpcrt4.def | 36 +- mingw-w64-crt/libarm32/rpcrt4.def | 534 - 2 files changed, 31 insertions(+), 539 deletions(-) rename mingw-w64-crt/{lib64 => lib-common

[Mingw-w64-public] [PATCH 00/27] More sharing and unification of def files

2017-08-17 Thread Martin Storsjö
. A header file is added that contains macros that allows specifying functions (or parts of declarations) that only should be included on some architectures. Martin Storsjö (27): crt: Include the def file in libwindowscodecs.a crt: Add a framework for sharing def files with arch specific

[Mingw-w64-public] [PATCH 19/27] crt: Unify ole32.def for lib64 and libarm32

2017-08-17 Thread Martin Storsjö
Base the new one on the libarm32 one, which has got more/newer functions. Include functions that seem to be specific to 64 bit mode (*_UserFree64, *_UserMarshal64, *_UserSize64, *_UserUnmarshal64) only when building for win64. Signed-off-by: Martin Storsjö <mar...@martin.st> --- mingw-w

[Mingw-w64-public] [PATCH 25/27] crt: Unify ws2_32.def for lib64 and libarm32

2017-08-17 Thread Martin Storsjö
Keep 64 bit specific functions (like WSCDeinstallProvider32) only when building for 64 bit mode. Signed-off-by: Martin Storsjö <mar...@martin.st> --- mingw-w64-crt/Makefile.am | 15 +- .../ws2_32.def => lib-common/ws2_32.def.in}| 23 ++- mingw-w64-

Re: [Mingw-w64-public] [PATCH 03/18] crt: Add an initial libarm64 directory with def files

2017-08-10 Thread Martin Storsjö
On Wed, 9 Aug 2017, Jacek Caban wrote: On 08.08.2017 22:32, Martin Storsjö wrote: The libarm64 directory is a copy of libarm32 with minimal modifications (renamings in the *.mri scripts and in Makefile.am). In that case I don't think we should have actual copies of every single .def file

[Mingw-w64-public] [PATCH 3/8] crt: Remove gzip.def from lib64 - this is not a system DLL

2017-08-10 Thread Martin Storsjö
This probably has been included by accident when dumping DLLs from the system directory. --- mingw-w64-crt/lib64/Makefile.am | 1 - mingw-w64-crt/lib64/gzip.def| 14 -- 2 files changed, 15 deletions(-) delete mode 100644 mingw-w64-crt/lib64/gzip.def diff --git

[Mingw-w64-public] [PATCH 4/8] crt: Remove nonsense def files for keybaord layout DLLs

2017-08-10 Thread Martin Storsjö
--- mingw-w64-crt/lib64/Makefile.am | 118 mingw-w64-crt/lib64/f3ahvoas.def| 11 mingw-w64-crt/lib64/kbd101.def | 10 --- mingw-w64-crt/lib64/kbd101a.def | 10 --- mingw-w64-crt/lib64/kbd101b.def | 10 ---

[Mingw-w64-public] [PATCH 1/8] crt: Include libarm32 def files in "make dist"

2017-08-10 Thread Martin Storsjö
--- mingw-w64-crt/Makefile.am | 3 +++ 1 file changed, 3 insertions(+) diff --git a/mingw-w64-crt/Makefile.am b/mingw-w64-crt/Makefile.am index b640e87..84b0e13 100644 --- a/mingw-w64-crt/Makefile.am +++ b/mingw-w64-crt/Makefile.am @@ -1534,10 +1534,13 @@ EXTRA_DIST = $(srcdir)/ChangeLog.*

[Mingw-w64-public] [PATCH 5/8] crt: Remove nonsense def files (LangDataCall)

2017-08-10 Thread Martin Storsjö
These only export LangDataCall as the only function. --- mingw-w64-crt/libarm32/Makefile.am | 4 mingw-w64-crt/libarm32/nl7data0011.def | 8 mingw-w64-crt/libarm32/nl7data001e.def | 8 mingw-w64-crt/libarm32/nl7data0404.def | 8

[Mingw-w64-public] [PATCH 8/8] crt: Share identical def files between lib64 and libarm32

2017-08-10 Thread Martin Storsjö
These def files are identical except for the header comment. --- mingw-w64-crt/Makefile.am | 7 + mingw-w64-crt/{lib64 => lib-common}/appmgmts.def | 0 mingw-w64-crt/{lib64 => lib-common}/audiosrv.def | 0 mingw-w64-crt/{lib64 => lib-common}/avrt.def | 0

[Mingw-w64-public] [PATCH 7/8] crt: Remove nonsense def files (plain DLL hooks)

2017-08-10 Thread Martin Storsjö
These files don't export any other functions than DLL hooks. --- mingw-w64-crt/lib32/Makefile.am | 1 - mingw-w64-crt/lib32/eapp3hst.def | 11 -- mingw-w64-crt/lib64/Makefile.am | 212 --- mingw-w64-crt/lib64/adrot.def| 12 ---

[Mingw-w64-public] [PATCH 6/8] crt: Remove nonsense def files (GetProxyDllInfo)

2017-08-10 Thread Martin Storsjö
All of these export a single symbol GetProxyDllInfo, plus some DLL hooks. --- mingw-w64-crt/lib64/Makefile.am | 15 -- mingw-w64-crt/lib64/actxprxy.def| 13 - mingw-w64-crt/lib64/catsrvps.def| 13 -

[Mingw-w64-public] [PATCH 2/8] crt: Split out the list of import libraries for lib64 into a separate Makefile.am

2017-08-10 Thread Martin Storsjö
This matches how it's already done for lib32 and libarm32. This makes it a little easier to modify the list of files with scripts. --- mingw-w64-crt/Makefile.am | 273 + mingw-w64-crt/lib64/Makefile.am | 1236 +++ 2 files changed, 1237

[Mingw-w64-public] [PATCH 1/3] crt: Remove def files that only contain a ServiceMain function

2017-08-10 Thread Martin Storsjö
--- mingw-w64-crt/lib64/6to4svc.def | 9 - mingw-w64-crt/lib64/Makefile.am | 15 --- mingw-w64-crt/lib64/aelupsvc.def | 9 - mingw-w64-crt/lib64/bthserv.def | 9 - mingw-w64-crt/lib64/dmserver.def

[Mingw-w64-public] [PATCH 2/3] crt: Remove def files that only contain a CreateInstance function

2017-08-10 Thread Martin Storsjö
--- mingw-w64-crt/libarm32/Makefile.am | 9 - mingw-w64-crt/libarm32/colorcnv.def| 8 mingw-w64-crt/libarm32/mp3dmod.def | 8 mingw-w64-crt/libarm32/mp4sdecd.def| 8 mingw-w64-crt/libarm32/resampledmo.def | 8

[Mingw-w64-public] [PATCH 3/3] crt: Remove def files that only contain InitHelperDll

2017-08-10 Thread Martin Storsjö
--- mingw-w64-crt/lib64/Makefile.am | 6 -- mingw-w64-crt/lib64/mon.def | 9 - mingw-w64-crt/lib64/dgnet.def | 13 - mingw-w64-crt/lib64/dhcpmon.def | 9 - mingw-w64-crt/lib64/hnetmon.def | 9 -

  1   2   3   4   5   6   7   8   9   10   >