[Bug ada/23492] [4.1 Regression] ACATS c48009e SEGV in set_bb_for_stmt tree-cfg.c:2673 on x86_64

[Bug ada/23492] [4.1 Regression] ACATS c48009e SEGV in set_bb_for_stmt tree-cfg.c:2673 on x86_64

[pinskia at gcc dot gnu dot org]

-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |4.1.0


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

[Bug middle-end/23492] [4.1 Regression] ACATS c48009e SEGV in set_bb_for_stmt tree-cfg.c:2673 on x86_64

[pinskia at gcc dot gnu dot org]

-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
          Component|ada                         |middle-end
           Priority|P2                          |P3


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

[Bug middle-end/23492] [4.1 Regression] ACATS c48009e SEGV in set_bb_for_stmt tree-cfg.c:2673 on x86_64

[pinskia at gcc dot gnu dot org]

------- Comment #1 from pinskia at gcc dot gnu dot org  2005-10-31 00:26 -------
Confirmed, this is a tree-inliner bug, we have a zero sized type which we are
inlining and we remove the statement and set the expression to NULL.  I have a
fix which I am going to bootstrap/test and apply as obvious if it passes.


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |pinskia at gcc dot gnu dot
                   |                            |org
         AssignedTo|unassigned at gcc dot gnu   |pinskia at gcc dot gnu dot
                   |dot org                     |org
             Status|UNCONFIRMED                 |ASSIGNED
     Ever Confirmed|0                           |1
   Last reconfirmed|0000-00-00 00:00:00         |2005-10-31 00:26:07
               date|                            |


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

[Bug middle-end/23492] [4.1 Regression] ACATS c48009e SEGV in set_bb_for_stmt tree-cfg.c:2673 on x86_64

[pinskia at gcc dot gnu dot org]

------- Comment #2 from pinskia at gcc dot gnu dot org  2005-10-31 05:02 -------
Here is a C testcase (so that Mark does not lower this to P5):
struct f {};
struct g1 {struct f l;};
static inline void g(struct f a, int i){}
void h(void)
{
  struct g1 t;
  g(t.l , 1);
}


-- 


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

[Bug middle-end/23492] [4.1 Regression] ACATS c48009e SEGV in set_bb_for_stmt tree-cfg.c:2673 on x86_64

[pinskia at gcc dot gnu dot org]

------- Comment #3 from pinskia at gcc dot gnu dot org  2005-10-31 05:13 -------
I decided against committing the patch as obvious.
Anyways the patch was posted:
http://gcc.gnu.org/ml/gcc-patches/2005-10/msg01770.html


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                URL|                            |http://gcc.gnu.org/ml/gcc-
                   |                            |patches/2005-
                   |                            |10/msg01770.html
           Keywords|                            |patch


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

[Bug middle-end/23492] [4.1 Regression] ACATS c48009e SEGV in set_bb_for_stmt tree-cfg.c:2673 on x86_64

[mmitchel at gcc dot gnu dot org]

------- Comment #4 from mmitchel at gcc dot gnu dot org  2005-10-31 05:22 -------
This patch is OK, assuming no objections within 24 hours.


-- 

mmitchel at gcc dot gnu dot org changed:

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


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

[Bug middle-end/23492] [4.1 Regression] ACATS c48009e SEGV in set_bb_for_stmt tree-cfg.c:2673 on x86_64

[pinskia at gcc dot gnu dot org]

------- Comment #6 from pinskia at gcc dot gnu dot org  2005-10-31 13:22 -------
Fixed.


-- 

pinskia at gcc dot gnu dot org changed:

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


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

[Bug middle-end/23492] [4.1 Regression] ACATS c48009e SEGV in set_bb_for_stmt tree-cfg.c:2673 on x86_64

[pinskia at gcc dot gnu dot org]

------- Comment #5 from pinskia at gcc dot gnu dot org  2005-10-31 13:22 -------
Subject: Bug 23492

Author: pinskia
Date: Mon Oct 31 13:22:20 2005
New Revision: 106270

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=106270
Log:
2005-10-31  Andrew Pinski  <pinskia@physics.uc.edu>

        PR middle-end/23492
        * tree-inline.c (setup_one_parameter): If the init_stmt
        is NULL, don't insert the statement.
2005-10-31  Andrew Pinski  <pinskia@physics.uc.edu>

        PR middle-end/23492
        * gcc.c-torture/compile/zero-strct-5.c: New test.


Added:
    trunk/gcc/testsuite/gcc.c-torture/compile/zero-strct-5.c
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/testsuite/ChangeLog
    trunk/gcc/tree-inline.c


-- 


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