Re: [Mingw-w64-public] [PATCH] Various changes for ARM

2014-05-30 19:04 GMT+02:00 André Hentschel : > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA1 > > Please review, i'll split it up and commit it. Hallo André, patch is ok. Please apply. Thanks, Kai -- Time is money. St

[Mingw-w64-public] [PATCH] Various changes for ARM

-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Please review, i'll split it up and commit it. -BEGIN PGP SIGNATURE- Version: GnuPG v1.4.12 (GNU/Linux) Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/ iQGcBAEBAgAGBQJTiLoLAAoJEGm5GZTakYsszxoMALMK9wIyw737hP/+HPbnj2VJ KZ1r4j49

Re: [Mingw-w64-public] [PATCH] Various changes for ARM

Hello André, The patch is fine beside one thing: Index: mingw-w64-crt/profile/profil.c === --- mingw-w64-crt/profile/profil.c (Revision 6627) +++ mingw-w64-crt/profile/profil.c (Arbeitskopie) @@ -43,9 +43,11 @@ ctx.ContextFlags =

[Mingw-w64-public] [PATCH] Various changes for ARM

Please review, i'll split it up and commit it. Index: mingw-w64-headers/crt/intrin.h === --- mingw-w64-headers/crt/intrin.h (Revision 6627) +++ mingw-w64-headers/crt/intrin.h (Arbeitskopie) @@ -1057,7 +1057,7 @@ /* __MACHINEI(__MI

Re: [Mingw-w64-public] [PATCH] Various changes for ARM

Patch is fine beside one nit. Line '+#elif defined(_ARM_)' should be '+#elif defined(_ARM_) || defined (__arm__)' With that patch is ok. Thanks, Kai 2014-05-09 20:28 GMT+02:00 André Hentschel : > What about this? > > Am 09.05.2014 00:08, schrieb Kai Tietz: >> Hi >> >> I would prefer here to use

Re: [Mingw-w64-public] [PATCH] Various changes for ARM

What about this? Am 09.05.2014 00:08, schrieb Kai Tietz: > Hi > > I would prefer here to use __x86_64__ __i386__ etc. > Have you tested that your changes work for IA targets? > > Cheers > Kai > > Am 08.05.2014 23:35 schrieb "André Hentschel" >: > > Please review,

Re: [Mingw-w64-public] [PATCH] Various changes for ARM

Is this meant to run on ia64?: > #if defined(_AMD64_) > movq(%rdx), %rax > movq8(%rdx), %rdx > xorq$0x8000, %rdx > movq%rax, (%rcx) > movq%rdx, 8(%rcx) > movq%rcx, %rax > ret Am 09.05.2014 00:08, schrieb Kai Tietz: > Hi > > I wou

Re: [Mingw-w64-public] [PATCH] Various changes for ARM

Hi I would prefer here to use __x86_64__ __i386__ etc. Have you tested that your changes work for IA targets? Cheers Kai Am 08.05.2014 23:35 schrieb "André Hentschel" : > Please review, i'll split it up and commit it. > > the tab before .globl is needed, otherwise it's not recognized on ARM. > D

[Mingw-w64-public] [PATCH] Various changes for ARM

Please review, i'll split it up and commit it. the tab before .globl is needed, otherwise it's not recognized on ARM. DECLSPEC_ALIGN(8) for the context seems to be hardly supported by our current toolchain Index: mingw-w64-crt/math/_chgsignl.S =