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
- From: "H.J. Lu" <hjl dot tools at gmail dot com>
- To: Kai Tietz <ktietz70 at googlemail dot com>
- Cc: JonY <jon_y at users dot sourceforge dot net>, Paolo Bonzini <bonzini at gnu dot org>, Dave Korn <dave dot korn dot cygwin at gmail dot com>, Gcc Patch List <gcc-patches at gcc dot gnu dot org>
- Date: Tue, 7 Sep 2010 07:06:47 -0700
- Subject: Re: [PATCH] Allow decimal floats for mingw-w64
- References: <4C73D99C.5050205@users.sourceforge.net> <AANLkTimRryeovwSU23BR3v8_X43mcNFMSNBsdo+v145Z@mail.gmail.com> <4C8040D2.90002@gmail.com> <4C803F00.5080500@users.sourceforge.net> <AANLkTik086--4UeuJQFiR06V8wHv9-p-tTJREYP0sJXN@mail.gmail.com>
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?
--
H.J.