This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [PATCH 5/8] Detect a frameless frame by comparing the FP register to -1.
- From: John Baldwin <jhb at freebsd dot org>
- To: Yao Qi <qiyaoltc at gmail dot com>
- Cc: "gdb-patches at sourceware dot org" <gdb-patches at sourceware dot org>, binutils at sourceware dot org
- Date: Tue, 14 Jun 2016 11:46:39 -0700
- Subject: Re: [PATCH 5/8] Detect a frameless frame by comparing the FP register to -1.
- Authentication-results: sourceware.org; auth=none
- References: <1465678115-58170-1-git-send-email-jhb at FreeBSD dot org> <1465678115-58170-6-git-send-email-jhb at FreeBSD dot org> <CAH=s-PPAjOdQkctdxcOWXoBeNNhcyT=4iR4k=9RRKFy0jb+_LA at mail dot gmail dot com>
On Monday, June 13, 2016 12:22:41 PM Yao Qi wrote:
> On Sat, Jun 11, 2016 at 9:48 PM, John Baldwin <jhb@freebsd.org> wrote:
> >
> > - if (cache->saved_regs[MEDIA_FP_REGNUM] >= 0)
> > + if (cache->saved_regs[MEDIA_FP_REGNUM] == -1)
> > cache->uses_fp = 1;
>
> I suspect it should be " != -1". saved_regs[MEDIA_FP_REGNUM] is initialized
> to -1, so if it is not the initialized value (-1), FP should be set in
> the prologue, and mark the flag uses_fp.
Yes, you are correct. Alternatively, we could change this code to explicitly
set 'cache->uses_fp' in the cases earlier in this function where it sets
MEDIA_FP_REGNUM to a value. sh-tdep.c follows this model, but it only needs
to set uses_fp in a few places.
--
John Baldwin