[Bug tree-optimization/24888] New: duplication of local variables after versioning/inlining

[Bug tree-optimization/24888] New: duplication of local variables after versioning/inlining

[razya at il dot ibm dot com]

For the testcase:
/home/razya/mainline_new_3/gcc/gcc/testsuite/gcc.c-torture/execute/20021111-1.c,
when using fipa-cp, a new vfersion is created
for aim_callhandler(). the static variable is copied twice into the
unexpanded_var_list of the versioned function. 
Looking at this test, even without enabling ipa-cp, the local variables
multiplies after inlining.(as shown in the dump below)
For example :
in 20021111-1.c.t23.alias1:

aim_callhandler (sess, conn, family, type)
{
  int D.1449;
  int sess;
  int conn;
  short unsigned int family;
  short unsigned int type;
  static int i = 0;
  int D.1457;
  int D.1456;
  int i.0;
  int D.1458;
  int D.1449;
  int D.1435;
  int sess;
  int conn;
  short unsigned int family;
  short unsigned int type;
  static int i = 0;
  int D.1443;
  int D.1442;
  int i.0;
  int D.1444;
  int D.1435;
  int D.1421;
  int sess;
  int conn;
  short unsigned int family;
  short unsigned int type;
  static int i = 0;
  int D.1429;
  int D.1428;
  int i.0;
  int D.1430;
  int D.1421;
  int D.1407;
  int sess;
  int conn;
  short unsigned int family;
  short unsigned int type;
  static int i = 0;
  int D.1415;
  int D.1414;
  int i.0;


-- 
           Summary: duplication of local variables after versioning/inlining
           Product: gcc
           Version: 4.1.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: tree-optimization
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: razya at il dot ibm dot com
 GCC build triplet: powerpc-suse-linux
  GCC host triplet:  powerpc-suse-linux
GCC target triplet: powerpc-suse-linux


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

[Bug tree-optimization/24888] duplication of local variables after versioning/inlining

[rguenth at gcc dot gnu dot org]

------- Comment #1 from rguenth at gcc dot gnu dot org  2005-11-16 11:46 -------
try dumping with -fdump-tree-*-uid, they are probably different copies.  You
can
try http://gcc.gnu.org/ml/gcc-patches/2005-08/msg00314.html to avoid having
unused versions dumped.


-- 


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

[Bug tree-optimization/24888] duplication of local variables after versioning/inlining

[razya at il dot ibm dot com]

------- Comment #2 from razya at il dot ibm dot com  2005-11-16 12:45 -------
(In reply to comment #1)
> try dumping with -fdump-tree-*-uid, they are probably different copies.  You
> can
> try http://gcc.gnu.org/ml/gcc-patches/2005-08/msg00314.html to avoid having
> unused versions dumped.

It doesn't help.
The decls are actually inserted to unexpanded_var_list more than once by the
inliner. Thanks.


-- 


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

[Bug tree-optimization/24888] duplication of local variables after versioning/inlining

[hubicka at gcc dot gnu dot org]

------- Comment #3 from hubicka at gcc dot gnu dot org  2006-07-22 13:28 -------
Testing patch


-- 

hubicka at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |hubicka at gcc dot gnu dot
                   |dot org                     |org
             Status|UNCONFIRMED                 |ASSIGNED
     Ever Confirmed|0                           |1
   Last reconfirmed|0000-00-00 00:00:00         |2006-07-22 13:28:08
               date|                            |


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

[Bug tree-optimization/24888] duplication of local variables after versioning/inlining

[patchapp at dberlin dot org]

------- Comment #4 from patchapp at dberlin dot org  2006-07-22 22:50 -------
Subject: Bug number PR tree-optimization/24888

A patch for this bug has been added to the patch tracker.
The mailing list url for the patch is
http://gcc.gnu.org/ml/gcc-patches/2006-07/msg00981.html


-- 


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

[Bug tree-optimization/24888] duplication of local variables after versioning/inlining

[hubicka at gcc dot gnu dot org]

------- Comment #5 from hubicka at gcc dot gnu dot org  2006-08-04 17:03 -------
Subject: Bug 24888

Author: hubicka
Date: Fri Aug  4 17:03:32 2006
New Revision: 115927

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=115927
Log:
        PR tree-optimization/24888
        * tree-inline.c (expand_call_inline): Do not re-record variables.
        (declare_inline_vars): Add variable to unexpanded_var_list.

Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/tree-inline.c


-- 


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

[Bug tree-optimization/24888] duplication of local variables after versioning/inlining

[bonzini at gnu dot org]

------- Comment #7 from bonzini at gnu dot org  2006-08-05 08:01 -------
fixed.


-- 

bonzini at gnu dot org changed:

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


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

[Bug tree-optimization/24888] duplication of local variables after versioning/inlining

[bonzini at gnu dot org]

------- Comment #6 from bonzini at gnu dot org  2006-08-05 08:00 -------
patch applied, fixed?


-- 


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

[Bug tree-optimization/24888] duplication of local variables after versioning/inlining

[patchapp at dberlin dot org]

------- Comment #8 from patchapp at dberlin dot org  2006-08-20 07:35 -------
Subject: Bug number PR tree-optimization/24888

A patch for this bug has been added to the patch tracker.
The mailing list url for the patch is
http://gcc.gnu.org/ml/gcc-patches/2006-08/msg00717.html


-- 


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