From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([140.186.70.92]:60527)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <asmwarrior@gmail.com>) id 1RHcRO-0001RU-0j
	for qemu-devel@nongnu.org; Sat, 22 Oct 2011 10:20:50 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <asmwarrior@gmail.com>) id 1RHcRM-0008Pe-O8
	for qemu-devel@nongnu.org; Sat, 22 Oct 2011 10:20:49 -0400
Received: from mail-iy0-f173.google.com ([209.85.210.173]:36830)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <asmwarrior@gmail.com>) id 1RHcRM-0008PU-L2
	for qemu-devel@nongnu.org; Sat, 22 Oct 2011 10:20:48 -0400
Received: by iakl21 with SMTP id l21so6883378iak.4
	for <qemu-devel@nongnu.org>; Sat, 22 Oct 2011 07:20:47 -0700 (PDT)
Message-ID: <4EA2D1EB.4060106@gmail.com>
Date: Sat, 22 Oct 2011 22:23:39 +0800
From: asmwarrior <asmwarrior@gmail.com>
MIME-Version: 1.0
References: <4E9BB180.6080506@mc.net>	<4E9C0497.2000605@siriusit.co.uk>	<4E9C3703.3040109@mc.net>	<4E9C645A.5060200@twiddle.net>	<4E9C9C08.20001@mc.net>	<CAEwic4Yw0nc9yij+SGMdXMy2R11FyXVWQ4TChczQsNGuVFqkAQ@mail.gmail.com>	<4E9CAACE.4070804@mc.net>	<CAEwic4aADQUtZOmjAYVVrS1cBU=uRNjwAV49f+GtPtro664JDw@mail.gmail.com>	<4E9F3BB4.3050604@mc.net>	<CA+BR0wxgvJ8=u5YtZRj7wSq=y6tafSFCHWV6-Zv3hsNbq3q6ZA@mail.gmail.com>	<CAEwic4ZJfwJBUgA2=Vwh3GBzCfqkTavYvXrZO7aicy=snVkQzg@mail.gmail.com>
	<CAEwic4Zey2f6fzSOAiGtyxuBnN2zSPwQCcDdTDdL8-DOiqC2GQ@mail.gmail.com>
In-Reply-To: <CAEwic4Zey2f6fzSOAiGtyxuBnN2zSPwQCcDdTDdL8-DOiqC2GQ@mail.gmail.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Subject: Re: [Qemu-devel] gcc auto-omit-frame-pointer vs msvc longjmp
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Kai Tietz <ktietz70@googlemail.com>
Cc: Mark Cave-Ayland <mark.cave-ayland@siriusit.co.uk>, "gcc@gcc.gnu.org" <gcc@gcc.gnu.org>, qemu-devel <qemu-devel@nongnu.org>, xunxun <xunxun1982@gmail.com>, Bob Breuer <breuerr@mc.net>, Richard Henderson <rth@twiddle.net>

On 2011-10-20 23:34, Kai Tietz wrote:
> Hi,
> 
> For trunk-version I have a tentative patch for this issue.  On 4.6.x
> and older branches this doesn't work, as here we can't differenciate
> that easy between ms- and sysv-abi.
> 
> But could somebody give this patch a try?
> 
> Regards,
> Kai
> 
> ChangeLog
> 
>          * config/i386/i386.c (ix86_frame_pointer_required): Enforce use of
>          frame-pointer for 32-bit ms-abi, if setjmp is used.
> 
> Index: i386.c
> ===================================================================
> --- i386.c      (revision 180099)
> +++ i386.c      (working copy)
> @@ -8391,6 +8391,10 @@
>     if (SUBTARGET_FRAME_POINTER_REQUIRED)
>       return true;
> 
> +  /* For older 32-bit runtimes setjmp requires valid frame-pointer.  */
> +  if (TARGET_32BIT_MS_ABI&&  cfun->calls_setjmp)
> +    return true;
> +
>     /* In ix86_option_override_internal, TARGET_OMIT_LEAF_FRAME_POINTER
>        turns off the frame pointer by default.  Turn it back on now if
>        we've not got a leaf function.  */
> 

I just see two related links about this issue:

see:
http://stackoverflow.com/questions/5887552/setjmp-and-omit-frame-pointer

and

http://connect.microsoft.com/VisualStudio/feedback/details/666704/visual-c-generates-incorrect-code-with-omit-frame-pointer-and-setjmp

I'm not sure microsoft has fix this issue.

asmwarrior
ollydbg from codeblocks' forum