Re: [PATCH] Allow decimal floats for mingw-w64

["H.J. Lu" <hjl dot tools at gmail dot com>]

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?


-- 
H.J.