This is the mail archive of the gcc-bugs@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

c++/7459: Missing '*' in unwind-dw2.c ?


>Number:         7459
>Category:       c++
>Synopsis:       Missing '*' in unwind-dw2.c ?
>Confidential:   no
>Severity:       non-critical
>Priority:       medium
>Responsible:    unassigned
>State:          open
>Class:          wrong-code
>Submitter-Id:   net
>Arrival-Date:   Wed Jul 31 13:16:00 PDT 2002
>Closed-Date:
>Last-Modified:
>Originator:     Mark Mendell
>Release:        gcc 3.1
>Organization:
>Environment:
Linux/PPC
>Description:
In routine uw_update_context_1, there is code:
    switch (fs->regs.reg[i].how)
      {
      case REG_UNSAVED:
        break;
      case REG_SAVED_OFFSET:
        context->reg[i] = cfa + fs->regs.reg[i].loc.offset;
        break;
      ...

Should the case for REG_SAVED_OFFSET be indirecting the value?  It seems that you are saving the address of the value, not the value itself.  This is inconsistent with the
other cases in the switch.
>How-To-Repeat:

>Fix:

>Release-Note:
>Audit-Trail:
>Unformatted:


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]