[Bug tree-optimization/43141] New: [4.5 Regression] Wrong debug information with IPA-SRA

[Bug tree-optimization/43141] New: [4.5 Regression] Wrong debug information with IPA-SRA

[rguenth at gcc dot gnu dot org]

Split out from PR42336.

But here's a C testcase where we lose debug info about a parameter that was
optimized away: there are no formal parameters in the DWARF output for f.

int i;
static int f(int) __attribute ((noinline));
static int f(int x) { return i; }

int main()
{
  return f(42);
}


-- 
           Summary: [4.5 Regression] Wrong debug information with IPA-SRA
           Product: gcc
           Version: 4.5.0
            Status: UNCONFIRMED
          Keywords: wrong-debug
          Severity: normal
          Priority: P3
         Component: tree-optimization
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: rguenth at gcc dot gnu dot org


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43141

[Bug tree-optimization/43141] [4.5 Regression] Wrong debug information with IPA-SRA

[rguenth at gcc dot gnu dot org]

------- Comment #1 from rguenth at gcc dot gnu dot org  2010-02-22 13:17 -------
Martin, http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42336#c22 still applies
here.


-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jamborm at gcc dot gnu dot
                   |                            |org
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1
   Last reconfirmed|0000-00-00 00:00:00         |2010-02-22 13:17:54
               date|                            |
   Target Milestone|---                         |4.5.0


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43141

[Bug tree-optimization/43141] [4.5 Regression] Wrong debug information with IPA-SRA

[jamborm at gcc dot gnu dot org]

------- Comment #2 from jamborm at gcc dot gnu dot org  2010-02-22 18:01 -------
Thanks for filing this separate PR.  I am aware of this problem and it is quite
high on my todo list.  My first attempts to address this however don't
bootstrap. I would like to get back to this by the end of the week but I think
next week is a more realistic estimate. 


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43141

[Bug tree-optimization/43141] [4.5 Regression] Wrong debug information with IPA-SRA

[rguenth at gcc dot gnu dot org]

-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P3                          |P1


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43141

[Bug tree-optimization/43141] [4.5 Regression] Wrong debug information with IPA-SRA

[jamborm at gcc dot gnu dot org]

-- 

jamborm at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |jamborm at gcc dot gnu dot
                   |dot org                     |org
             Status|NEW                         |ASSIGNED
   Last reconfirmed|2010-02-22 13:17:54         |2010-03-08 11:14:17
               date|                            |


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43141

[Bug tree-optimization/43141] [4.5 Regression] Wrong debug information with IPA-SRA

[jamborm at gcc dot gnu dot org]

------- Comment #3 from jamborm at gcc dot gnu dot org  2010-03-10 16:36 -------
Created an attachment (id=20075)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20075&action=view)
Potential (preliminary) fix

After much tinkering I have came up with this patch do create an
ABSTRACT_ORIGIN for functions affected by IPA-SRA.  The formal
parameter x is in debug info as shown by dwarfdump.  The C and C++
testsuites have shown no regressions and the full bootstrap I am
running at the moment is already in stage 3.

However, this patch certainly does cut some corners so if you think it
is unsuitable, please let me know, the sooner the better.  Otherwise,
if all goes well I'll add the missing comments, changelog,
dependencies and stuff and will submit the patch for trunk.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43141

[Bug tree-optimization/43141] [4.5 Regression] Wrong debug information with IPA-SRA

[jamborm at gcc dot gnu dot org]

------- Comment #4 from jamborm at gcc dot gnu dot org  2010-03-15 12:49 -------
Fixed by:

r157456 | jamborm | 2010-03-15 12:43:04 +0100 (Mon, 15 Mar 2010) | 7 lines

2010-03-15  Martin Jambor  <mjambor@suse.cz>

        PR tree/optimization/43141
        * tree-sra.c (create_abstract_origin): New function.
        (modify_function): Call create_abstract_origin.


-- 

jamborm at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43141

[Bug tree-optimization/43141] [4.5 Regression] Wrong debug information with IPA-SRA

[jamborm at gcc dot gnu dot org]

------- Comment #5 from jamborm at gcc dot gnu dot org  2010-04-01 13:32 -------
Subject: Bug 43141

Author: jamborm
Date: Thu Apr  1 13:31:53 2010
New Revision: 157909

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=157909
Log:
2010-04-01  Martin Jambor  <mjambor@suse.cz>

        PR tree-optimization/43141
        * testsuite/gcc.dg/guality/pr43141.c: New test.


Added:
    trunk/gcc/testsuite/gcc.dg/guality/pr43141.c
Modified:
    trunk/gcc/testsuite/ChangeLog


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43141