X-Recipient: archive-cygwin AT delorie DOT com
DomainKey-Signature: a=rsa-sha1; c=nofws; d=sourceware.org; h=list-id
	:list-unsubscribe:list-subscribe:list-archive:list-post
	:list-help:sender:mime-version:content-type
	:content-transfer-encoding:date:from:to:subject:in-reply-to
	:references:message-id; q=dns; s=default; b=EgWwocCk0bzVPM2GuwUg
	YuxnwxFqLgksAeqQXqQALGfbsJvJ9IHIP3D+I48CfksRQqOPuZ+I3LF18TfC1JR3
	C5g6od+MgY/ZNM4Yn3w8VUPtaLzyKKFmBZ5Fy7znKbjXXmWWr+yJQiufox1ZxvRJ
	87o5y6X4pNystzvkBCzZRD0=
DKIM-Signature: v=1; a=rsa-sha1; c=relaxed; d=sourceware.org; h=list-id
	:list-unsubscribe:list-subscribe:list-archive:list-post
	:list-help:sender:mime-version:content-type
	:content-transfer-encoding:date:from:to:subject:in-reply-to
	:references:message-id; s=default; bh=yD5bhPzueFW3H62fEYAtnlZl7z
	0=; b=g3HHr2fM3KLACuATpLa0GbXbHHdqzsEjQd9xaC7gvhSZEWD6LUJ66GMZj1
	ADhs3bowCPx+1xXiTgzeYzhDmU0w/ItGGPYY4Tn2YMMDE51Qc096wiwIz0towjjD
	Emxhk9QtMeAv5Lf0FpYs77Y9rjNKJZpf4FaduZULlRTKgI2sA=
Mailing-List: contact cygwin-help AT cygwin DOT com; run by ezmlm
List-Id: <cygwin.cygwin.com>
List-Subscribe: <mailto:cygwin-subscribe AT cygwin DOT com>
List-Archive: <http://sourceware.org/ml/cygwin/>
List-Post: <mailto:cygwin AT cygwin DOT com>
List-Help: <mailto:cygwin-help AT cygwin DOT com>, <http://sourceware.org/ml/#faqs>
Sender: cygwin-owner AT cygwin DOT com
Mail-Followup-To: cygwin AT cygwin DOT com
Delivered-To: mailing list cygwin AT cygwin DOT com
Authentication-Results: sourceware.org; auth=none
X-Spam-SWARE-Status: No, score=-2.5 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_LOW,SPF_PASS autolearn=ham version=3.3.2 spammy=Hx-spam-relays-external:ESMTPA
X-HELO: lb3-smtp-cloud8.xs4all.net
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII; format=flowed
Content-Transfer-Encoding: 7bit
Date: Thu, 02 Aug 2018 17:14:52 +0200
From: Houder <houder AT xs4all DOT nl>
To: cygwin AT cygwin DOT com
Subject: Re: x86_64: floating-point environment (i.e. fenv.h). BUG.
In-Reply-To: <4438ec88ad4e6965281be138d93ba45d@xs4all.nl>
References: <3a882a7d38c52dc5a14d176f30f8340e AT xs4all DOT nl> <20180801110009 DOT GA30124 AT calimero DOT vinschen DOT de> <4438ec88ad4e6965281be138d93ba45d AT xs4all DOT nl>
Message-ID: <76343c2b9f4a4bbb8bece5150098733a@xs4all.nl>
X-Sender: houder AT xs4all DOT nl
User-Agent: XS4ALL Webmail
X-IsSubscribed: yes

On 2018-08-01 13:49, Houder wrote:
> On 2018-08-01 13:00, Corinna Vinschen wrote:
>> On Aug  1 12:22, Houder wrote:
>>> Hi Corinna,
>>> 
>>> Short version of my report (as there is more to say about the 
>>> implementation
>>> of
>>> "fenv") in Cygwin; this time I restrict myself to a bug in fegetenv() 
>>> ).
>>> 
>>> (Note to myself: attach STC)
>>> 
>>> I am reporting a bug in fegetenv() in winsup/cygwin/fenv.cc. There is 
>>> no
>>> hurry
>>> in repairing this bug, as "fenv" is hardly ever (never?) used by 
>>> anyone.
>>> 
>>> fegetenv() should be modified as follows:
>>> 
>>> from:
>>>   __asm__ volatile ("fnstenv %0" : "=m" (envp->_fpu) : );
>>>   if (use_sse)
>>>     __asm__ volatile ("stmxcsr %0" : "=m" (envp->_sse_mxcsr) : );
>>>   return 0
>>> 
>>> to:
>>> // Henri: copying glibc ...
>>>   __asm__ volatile ("fnstenv %0\n"
>>>                     "fldenv %0" : "=m" (envp->_fpu) : );
>>>   if (use_sse)
>>>     __asm__ volatile ("stmxcsr %0" : "=m" (envp->_sse_mxcsr) : );
>>>   return 0;
>> 
>> Since you know how to fix things, please just send patches to
>> the cygwin-patches ML.
> 
> Ah ... Agreed. However, as I am not set up for building (anymore), this
> will take some time ... (not even git is installed).
> 
> (yes, I did verify the above modification (and more), however I did 
> that
>  "locally" (basically, I included winsup/cygwin/fenv.cc in my  STC) ).

Corinna,

As requested by you, I made an attempt. See cygwin-patches.

Regards,

Henri

--
Problem reports:       http://cygwin.com/problems.html
FAQ:                   http://cygwin.com/faq/
Documentation:         http://cygwin.com/docs.html
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple