This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Allow decimal floats for mingw-w64
On 9/7/2010 23:56, JonY wrote:
> On 9/7/2010 23:57, H.J. Lu wrote:
>> On Tue, Sep 7, 2010 at 8:29 AM, JonY<jon_y@users.sourceforge.net> wrote:
>>> On 9/7/2010 22:06, H.J. Lu wrote:
>>>>
>>>> On Tue, Sep 7, 2010 at 1:51 AM, Kai Tietz<ktietz70@googlemail.com>
>>>> wrote:
>>>>>
>>>>> 2010/9/3 JonY<jon_y@users.sourceforge.net>:
>>>>>>
>>>>>> On 9/3/2010 08:26, Dave Korn wrote:
>>>>>>>
>>>>>>> On 24/08/2010 16:06, Kai Tietz wrote:
>>>>>>>>
>>>>>>>> 2010/8/24 JonY<jon_y@users.sourceforge.net>:
>>>>>>>>>
>>>>>>>>> Hi,
>>>>>>>>>
>>>>>>>>> Attached patch enables i?86-w64-mingw32 and x86_64-w64-mingw32
>>>>>>>>> decimal
>>>>>>>>> float
>>>>>>>>> support with bid.
>>>>>>>>>
>>>>>>>>> Tested with x86_64-w64-mingw32. Is attached patch OK for trunk?
>>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>>>>>>>> 2010-08-24 Jonathan Yong<jon_y@users.sourceforge.net>
>>>>>>>>>
>>>>>>>>> * configure.ac: Allow w64 targets to use bid decimal floats.
>>>>>>>>> * configure: Regenerated.
>>>>>>>>>
>>>>>>>>>
>>>>>>>>
>>>>>>>> Patch is from my side ok. I added Dave to CC. Maybe he wants
>>>>>>>> that it
>>>>>>>> gets added for cygwin, too?
>>>>>>>
>>>>>>> I'm currently testing a build that I did with Uros' patch and
>>>>>>> manually
>>>>>>> configuring with '--enable-decimal-float' and it seems to be going
>>>>>>> pretty
>>>>>>> well, so yes please Jon, if you don't mind spinning another respin.
>>>>>>>
>>>>>>
>>>>>> Spinning another respin? I thought --enable-decimal-float was ignored
>>>>>> for cygwin and mingw?
>>>>>>
>>>>>>> BTW, re Ralf's earlier point: for preference please actually /omit/
>>>>>>> the
>>>>>>> diffs of the generated configure files altogether, we only need
>>>>>>> to see
>>>>>>> the
>>>>>>> change to the .ac file, because autoconf is very deterministic,
>>>>>>> and the
>>>>>>> actual
>>>>>>> configure file diffs often contain tons of uninformative noise
>>>>>>> from the
>>>>>>> expansions of all the macros and the changes in error line number
>>>>>>> reporting(*).
>>>>>>>
>>>>>>
>>>>>> Yes, I was wondering about that. New patch attached, this time only
>>>>>> configure.ac and the new m4 file.
>>>>>>
>>>>>
>>>>> Hello,
>>>>>
>>>>> Patch is ok. Thanks for reworking this but I would like that Paolo
>>>>> takes a look on it, too.
>>>>>
>>>>
>>>> Does this patch apply to trunk?
>>>>
>>>>
>>>
>>> Well, its supposed to be for gcc trunk.
>>>
>>> I did make another patch without the autoreconf stuff added, did it get
>>> lost?
>>>
>>
>> Where is it? Have you tried it on today's trunk?
>>
>>
>
> I seems somebody has already committed a somewhat related patch, I'll
> need to rework mine.
Looks like most of the changes are in, the only parts needed changing was:
Index: config/dfp.m4
===================================================================
--- config/dfp.m4 (revision 163959)
+++ config/dfp.m4 (working copy)
@@ -20,7 +20,9 @@
],
[
case $1 in
- powerpc*-*-linux* | i?86*-*-linux* | x86_64*-*-linux* | s390*-*-linux*)
+ powerpc*-*-linux* | i?86*-*-linux* | x86_64*-*-linux* | s390*-*-linux* | \
+ i?86*-*-mingw* | x86_64*-*-mingw* |\
+ i?86*-*-cygwin* )
enable_decimal_float=yes
;;
*)