Mirror of the gdb mailing list
 help / color / mirror / Atom feed
From: Daniel Jacobowitz <drow@false.org>
To: Eli Zaretskii <eliz@gnu.org>
Cc: gdb@sources.redhat.com, Reiner.Steib@gmx.de
Subject: Re: Variable "foo" is not available
Date: Mon, 04 Apr 2005 13:37:00 -0000	[thread overview]
Message-ID: <20050404133743.GA32163@nevyn.them.org> (raw)
In-Reply-To: <01c538d4$Blat.v2.4$b261c020@zahav.net.il>

On Mon, Apr 04, 2005 at 08:10:00AM +0300, Eli Zaretskii wrote:
> > Date: Sat, 2 Apr 2005 16:05:42 -0500
> > From: Daniel Jacobowitz <drow@false.org>
> > Cc: gdb@sources.redhat.com, Reiner.Steib@gmx.de
> > 
> > > We are talking about function call arguments here, not just about any
> > > local variables.  Can you tell what compiler optimizations could cause
> > > what Reiner reported: that the first argument is available to GDB, but
> > > the second is not?
> > 
> > Very easily.  Suppose you have two incoming arguments in registers; GCC
> > will do this automatically for static functions even on i386, which
> > normally uses a stack convention.  The first is used after a function
> > call, so it is preserved by saving it to the stack.  The second is not
> > used after the function call, so the compiler has no reason to allocate
> > a save slot for it, and no reason to store it to memory before the
> > function call.
> 
> The functions present in Reiner's backtraces are not static, they are
> external, with the exception of funcall_lambda.  I don't have access
> to an x86_64 machine, but at least on an IA32 x86 architecture the
> code produced by GCC 3.4.3 for these function calls is quite
> straightforward (see one example below), and with GDB 6.3 I couldn't
> reproduce the "arg not available" message.

He gave us the missing clue in a later message - as Mark wrote, on
x86_64, the arguments are in registers.  This means the compiler must
explicitly save them.

> > With stack-based argument passing, GCC may be claiming an argument is
> > unavailable when the function's local copy is dead, when a copy still
> > exists on the stack somewhere.  I don't know if it will do that or not.
> > GDB can not assume that the argument is available in the incoming stack
> > slot, since it could be reused for other data.
> 
> What, if any, would be the expression of this in the machine code?

My x86 assembly is awful, so I tried to derive this from gcc output. 
The version of GCC I have installed will generate debug information
referring to the incoming argument slot, which I didn't expect it to
do.  So this is probably a non-issue.

> Also, I don't quite understand how can a stack slot of a function call
> argument be reused before the function returns.  Isn't that slot
> outside the function's frame?  Reusing it would be a violation of the
> ABI, right?

Actually, I don't think it would be.  This has been the subject of
considerable debate on the linux-kernel list in the past; GCC will 
sometimes modify these slots and the final consesnsus was that it was
within its rights to do so.

int foo();
int foo2 (int *);
int bar(int a)
{
  foo ();
  a += 3;
  foo2 (&a);
  return a + foo();
}

   0:   55                      push   %ebp
   1:   89 e5                   mov    %esp,%ebp
   3:   83 ec 08                sub    $0x8,%esp
   6:   e8 fc ff ff ff          call   7 <bar+0x7>
   b:   83 45 08 03             addl   $0x3,0x8(%ebp)
   f:   8d 45 08                lea    0x8(%ebp),%eax
  12:   89 04 24                mov    %eax,(%esp)
  15:   e8 fc ff ff ff          call   16 <bar+0x16>
  1a:   e8 fc ff ff ff          call   1b <bar+0x1b>
  1f:   8b 55 08                mov    0x8(%ebp),%edx
  22:   89 ec                   mov    %ebp,%esp
  24:   5d                      pop    %ebp
  25:   01 d0                   add    %edx,%eax
  27:   c3                      ret

See the instruction at 0xb?

GCC won't reuse the slot for an unrelated variable at present. 
However, in the future, it would be a valid optimization.

-- 
Daniel Jacobowitz
CodeSourcery, LLC


  parent reply	other threads:[~2005-04-04 13:37 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-04-01 16:40 Reiner Steib
2005-04-01 17:19 ` Daniel Jacobowitz
2005-04-02  9:49   ` Eli Zaretskii
2005-04-02 13:53     ` Reiner Steib
2005-04-02 14:27       ` Daniel Jacobowitz
2005-04-06 16:25         ` Reiner Steib
2005-04-02 14:26     ` Daniel Jacobowitz
2005-04-02 18:17       ` Eli Zaretskii
2005-04-02 18:40         ` Daniel Jacobowitz
2005-04-02 20:58           ` Eli Zaretskii
2005-04-02 21:05             ` Daniel Jacobowitz
2005-04-04  5:14               ` Eli Zaretskii
2005-04-04  6:00                 ` Mark Kettenis
2005-04-04  7:58                 ` Daniel THOMPSON
2005-04-04 19:28                   ` Eli Zaretskii
2005-04-04 13:37                 ` Daniel Jacobowitz [this message]
2005-04-04 19:35                   ` Eli Zaretskii
2005-04-04 19:41                     ` Daniel Jacobowitz
2005-04-03 18:16           ` Reiner Steib
2005-04-08 11:05       ` Eli Zaretskii
2005-04-04  9:26     ` Reiner Steib

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20050404133743.GA32163@nevyn.them.org \
    --to=drow@false.org \
    --cc=Reiner.Steib@gmx.de \
    --cc=eliz@gnu.org \
    --cc=gdb@sources.redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox