Mailing-List: contact cygwin-developers-help AT sourceware DOT cygnus DOT com; run by ezmlm List-Subscribe: List-Archive: List-Post: List-Help: , Sender: cygwin-developers-owner AT sources DOT redhat DOT com Delivered-To: mailing list cygwin-developers AT sources DOT redhat DOT com Date: Thu, 7 Sep 2000 23:11:11 -0400 From: Chris Faylor To: Chris Faylor Subject: Re: sigframe inlining Message-ID: <20000907231111.A715@cygnus.com> Reply-To: cygwin-developers AT sources DOT redhat DOT com Mail-Followup-To: Chris Faylor References: <1975989842 DOT 20000901235524 AT logos-m DOT ru> <20000901160904 DOT A29015 AT cygnus DOT com> <1137440989 DOT 20000902001935 AT logos-m DOT ru> <20000901163248 DOT A29171 AT cygnus DOT com> <1958635367 DOT 20000902003930 AT logos-m DOT ru> <20000901205807 DOT A32178 AT cygnus DOT com> <20000901211953 DOT A32424 AT cygnus DOT com> <11977784207 DOT 20000902195159 AT logos-m DOT ru> <20000903005142 DOT A15751 AT cygnus DOT com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.3.6i In-Reply-To: <20000903005142.A15751@cygnus.com>; from cgf@cygnus.com on Sun, Sep 03, 2000 at 12:51:42AM -0400 On Sun, Sep 03, 2000 at 12:51:42AM -0400, Chris Faylor wrote: >On Sat, Sep 02, 2000 at 07:51:59PM +0400, Egor Duda wrote: >>Hi! >> >>Saturday, 02 September, 2000 Chris Faylor cgf AT cygnus DOT com wrote: >> >>>>I've duplicated the problem. A test case is below. >>>> >>>>I'll look into this. >> >>CF> I've checked in a fix. >> >>Thanks! >> >>today i've got another bug, when i've compiled cygwin without -O2 flag >>since gcc stops inining class sigframe member functions, >>__builtin_frame_address got wrong stack offset. this leads to >>api_fatal in signal handler. how about this patch? >> >>Egor. mailto:deo AT logos-m DOT ru ICQ 5165414 FidoNet 2:5020/496.19 > >Yeah, I knew about this problem and have studiously avoided compiling >sigframe functions without optimization. I wish there was some way to >force inlining no matter what. > >I guess we'll probably have to go with something like this eventually >but I'd like to ruminate on it for a while first. I've checked in a patch that, I think, accomplishes the same thing without the use of additional macros. cgf