[Bug tree-optimization/30262] New: [4.1/4.2/4.3 Regression] ICE with nested fn accessed var in asm "m" constraint

[Bug tree-optimization/30262] New: [4.1/4.2/4.3 Regression] ICE with nested fn accessed var in asm "m" constraint

[jakub at gcc dot gnu dot org]

void
foo (void)
{
  unsigned int x = 0;

  void bar (void)
  {
    x = 254;
  }

  bar ();
  asm volatile ("" :: "m" (x));
}

ICEs in various places at -O and higher in both 4.1.1 and on the trunk.
>From initial skim, it seems tree-nested.c ought to be more careful when
changing
asms.


-- 
           Summary: [4.1/4.2/4.3 Regression] ICE with nested fn accessed var
                    in asm "m" constraint
           Product: gcc
           Version: 4.1.2
            Status: UNCONFIRMED
          Keywords: ice-on-valid-code
          Severity: normal
          Priority: P3
         Component: tree-optimization
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: jakub at gcc dot gnu dot org


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

[Bug tree-optimization/30262] [4.1/4.2/4.3 Regression] ICE with nested fn accessed var in asm "m" constraint

[pinskia at gcc dot gnu dot org]

------- Comment #1 from pinskia at gcc dot gnu dot org  2006-12-19 23:12 -------
OpenMP has the same problem, see PR 30263.


-- 


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

[Bug tree-optimization/30262] [4.1/4.2/4.3 Regression] ICE with nested fn accessed var in asm "m" constraint

[pinskia at gcc dot gnu dot org]

------- Comment #2 from pinskia at gcc dot gnu dot org  2006-12-19 23:13 -------
(In reply to comment #1)
> OpenMP has the same problem, see PR 30263.
This is most likely because most of the openmp lowering code was copied from
tree-nested.c :).


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |pinskia at gcc dot gnu dot
                   |                            |org
   Target Milestone|---                         |4.1.2


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

[Bug tree-optimization/30262] [4.0/4.1/4.2/4.3 Regression] ICE with nested fn accessed var in asm "m" constraint

[pinskia at gcc dot gnu dot org]

------- Comment #3 from pinskia at gcc dot gnu dot org  2006-12-19 23:17 -------
Here is a rejects valid which makes this a regression in 4.0.x also.
void
foo (void)
{
  unsigned int x = 0;

  void bar (void)
  {
    x = 254;
  }

  bar ();
  asm ("" : "=m" (x));
}


Confirmed.


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1
           Keywords|                            |rejects-valid
   Last reconfirmed|0000-00-00 00:00:00         |2006-12-19 23:17:56
               date|                            |
            Summary|[4.1/4.2/4.3 Regression] ICE|[4.0/4.1/4.2/4.3 Regression]
                   |with nested fn accessed var |ICE with nested fn accessed
                   |in asm "m" constraint       |var in asm "m" constraint
   Target Milestone|4.1.2                       |4.0.4


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

[Bug middle-end/30262] [4.0/4.1/4.2/4.3 Regression] ICE with nested fn accessed var in asm "m" constraint

[jakub at gcc dot gnu dot org]

-- 

jakub at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |jakub at gcc dot gnu dot org
                   |dot org                     |
             Status|NEW                         |ASSIGNED
   Last reconfirmed|2006-12-19 23:17:56         |2006-12-20 14:38:43
               date|                            |


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

[Bug middle-end/30262] [4.0/4.1/4.2/4.3 Regression] ICE with nested fn accessed var in asm "m" constraint

[jakub at gcc dot gnu dot org]

------- Comment #4 from jakub at gcc dot gnu dot org  2006-12-21 13:13 -------
Subject: Bug 30262

Author: jakub
Date: Thu Dec 21 13:13:15 2006
New Revision: 120106

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=120106
Log:
        PR middle-end/30262
        PR middle-end/30263
        * tree-nested.c (walk_asm_expr): New function.
        (walk_stmts): Use it for ASM_EXPR.

        * gcc.c-torture/execute/20061220-1.c: New test.
        * gcc.dg/gomp/asm-1.c: New test.

Added:
    trunk/gcc/testsuite/gcc.c-torture/execute/20061220-1.c
    trunk/gcc/testsuite/gcc.dg/gomp/asm-1.c
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/testsuite/ChangeLog
    trunk/gcc/tree-nested.c


-- 


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

[Bug middle-end/30262] [4.0/4.1/4.2/4.3 Regression] ICE with nested fn accessed var in asm "m" constraint

[jakub at gcc dot gnu dot org]

------- Comment #5 from jakub at gcc dot gnu dot org  2006-12-21 13:19 -------
Subject: Bug 30262

Author: jakub
Date: Thu Dec 21 13:18:52 2006
New Revision: 120107

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=120107
Log:
        PR middle-end/30262
        PR middle-end/30263
        * tree-nested.c (walk_asm_expr): New function.
        (walk_stmts): Use it for ASM_EXPR.

        * gcc.c-torture/execute/20061220-1.c: New test.
        * gcc.dg/gomp/asm-1.c: New test.

Added:
    branches/gcc-4_2-branch/gcc/testsuite/gcc.c-torture/execute/20061220-1.c
    branches/gcc-4_2-branch/gcc/testsuite/gcc.dg/gomp/asm-1.c
Modified:
    branches/gcc-4_2-branch/gcc/ChangeLog
    branches/gcc-4_2-branch/gcc/testsuite/ChangeLog
    branches/gcc-4_2-branch/gcc/tree-nested.c


-- 


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

[Bug middle-end/30262] [4.0/4.1/4.2/4.3 Regression] ICE with nested fn accessed var in asm "m" constraint

[jakub at gcc dot gnu dot org]

------- Comment #6 from jakub at gcc dot gnu dot org  2006-12-21 13:37 -------
Subject: Bug 30262

Author: jakub
Date: Thu Dec 21 13:37:41 2006
New Revision: 120108

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=120108
Log:
        PR middle-end/30262
        PR middle-end/30263
        * tree-nested.c (walk_asm_expr): New function.
        (walk_stmts): Use it for ASM_EXPR.

        * gcc.c-torture/execute/20061220-1.c: New test.
        * gcc.dg/gomp/asm-1.c: New test.

Added:
   
branches/redhat/gcc-4_1-branch/gcc/testsuite/gcc.c-torture/execute/20061220-1.c
    branches/redhat/gcc-4_1-branch/gcc/testsuite/gcc.dg/gomp/asm-1.c
Modified:
    branches/redhat/gcc-4_1-branch/gcc/ChangeLog
    branches/redhat/gcc-4_1-branch/gcc/testsuite/ChangeLog
    branches/redhat/gcc-4_1-branch/gcc/tree-nested.c


-- 


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

[Bug middle-end/30262] [4.0/4.1 Regression] ICE with nested fn accessed var in asm "m" constraint

[jakub at gcc dot gnu dot org]

------- Comment #7 from jakub at gcc dot gnu dot org  2006-12-21 13:53 -------
Fixed for 4.2/4.3 so far.


-- 

jakub at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|[4.0/4.1/4.2/4.3 Regression]|[4.0/4.1 Regression] ICE
                   |ICE with nested fn accessed |with nested fn accessed var
                   |var in asm "m" constraint   |in asm "m" constraint


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

[Bug middle-end/30262] [4.0/4.1 Regression] ICE with nested fn accessed var in asm "m" constraint

[gdr at gcc dot gnu dot org]

------- Comment #8 from gdr at gcc dot gnu dot org  2007-02-03 21:20 -------
won't fix in GCC-4.0.x.  Adjusting milestone.


-- 

gdr at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|4.0.4                       |4.1.3


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

[Bug middle-end/30262] [4.0/4.1 Regression] ICE with nested fn accessed var in asm "m" constraint

[pinskia at gcc dot gnu dot org]

-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|4.1.3                       |4.1.2


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

[Bug middle-end/30262] [4.0/4.1 Regression] ICE with nested fn accessed var in asm "m" constraint

[mmitchel at gcc dot gnu dot org]

------- Comment #9 from mmitchel at gcc dot gnu dot org  2007-02-05 05:48 -------
Jakub, you've checked in a patch on the 4.1 branch, but your comment says that
this is fixed for 4.2 and 4.3.  Was it also fixed for 4.1, or is there still a
problem on 4.1?

Thanks,

-- Mark


-- 

mmitchel at gcc dot gnu dot org changed:

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


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

[Bug middle-end/30262] [4.0/4.1 Regression] ICE with nested fn accessed var in asm "m" constraint

[jakub at gcc dot gnu dot org]

------- Comment #10 from jakub at gcc dot gnu dot org  2007-02-05 06:52 -------
Only on redhat/gcc-4_1-branch.  tree-nested.c on that branch is far closer
to gcc-4_2-branch (due to backported OpenMP support).  Backporting to
vanilla gcc-4_1-branch will need more work.


-- 


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

[Bug middle-end/30262] [4.0/4.1 Regression] ICE with nested fn accessed var in asm "m" constraint

[mmitchel at gcc dot gnu dot org]

-- 

mmitchel at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|4.1.2                       |4.1.3


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

[Bug middle-end/30262] [4.0/4.1 Regression] ICE with nested fn accessed var in asm "m" constraint

[rguenth at gcc dot gnu dot org]

------- Comment #11 from rguenth at gcc dot gnu dot org  2008-02-20 22:47 -------
Fixed since 4.2.0, wontfix on older branches.


-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
      Known to fail|                            |4.1.3
      Known to work|                            |4.2.0
         Resolution|                            |FIXED
   Target Milestone|4.1.3                       |4.2.0


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