Am 16.05.2016 um 15:58 schrieb Hugo Beauzée-Luyssen: > On 05/16/2016 03:55 PM, André Hentschel wrote: >> Am 13.05.2016 um 14:17 schrieb Kai Tietz: >>> Hi, >>> >>> from my point of view this is ok. Nevertheless I would like that >>> Andre is taking a look to it before. >>> >>> Thanks, >>> Kai >>> >> >> long double is the same as double on ARM afaicr, so I'm not sure how a build >> could break with that code. >> But the patch is also not wrong, but should be even more correct... >> > > I must say I don't know the arm specifics, and trust you if you're saying it > should behave the same. > The rational for this patch is simply this compiler output: > > /build/prefix/armv7-w64-mingw32/include/math.h:434:11: error: assigning to > 'double *' from incompatible type 'long double *' > hlp.d = &x; > ^ ~~ > /build/prefix/armv7-w64-mingw32/include/math.h:594:11: error: assigning to > 'double *' from incompatible type 'long double *' > hlp.d = &_x; > ^ ~~~ > /build/prefix/armv7-w64-mingw32/include/math.h:665:11: error: assigning to > 'double *' from incompatible type 'long double *' > hlp.d = &x; > ^ ~~ > > When building with llvm/clang. > > Regards,
OK, thx. I only tested with gcc... But I also just noticed that you should also change the return line dt to ldt or simply merge the implementation with the x86_64 one ------------------------------------------------------------------------------ Mobile security can be enabling, not merely restricting. Employees who bring their own devices (BYOD) to work are irked by the imposition of MDM restrictions. Mobile Device Manager Plus allows you to control only the apps on BYO-devices by containerizing them, leaving personal data untouched! https://ad.doubleclick.net/ddm/clk/304595813;131938128;j _______________________________________________ Mingw-w64-public mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/mingw-w64-public
