[Bug tree-optimization/22088] New: [4.1 regression] ICE with -ftree-vectorize

[Bug tree-optimization/22088] New: [4.1 regression] ICE with -ftree-vectorize

[reichelt at gcc dot gnu dot org]

The following testcase causes an ICE on i686-pc-linux-gnu when compiled with
"-O -march=pentium4 -ftree-vectorize":

===============================
double d[3];

void foo() 
{ 
    int i;
    for (i=0; i<3; ++i)
        d[i] = 0;
}
===============================

bug.c: In function 'foo':
bug.c:4: internal compiler error: in get_bb_copy, at cfg.c:1078
Please submit a full bug report, [etc.]

The regression was introduced yesterday (2005-06-15).

-- 
           Summary: [4.1 regression] ICE with -ftree-vectorize
           Product: gcc
           Version: 4.1.0
            Status: UNCONFIRMED
          Keywords: ice-on-valid-code, monitored
          Severity: normal
          Priority: P2
         Component: tree-optimization
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: reichelt at gcc dot gnu dot org
                CC: gcc-bugs at gcc dot gnu dot org


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

[Bug tree-optimization/22088] [4.1 regression] ICE with -ftree-vectorize

[reichelt at gcc dot gnu dot org]

------- Additional Comments From reichelt at gcc dot gnu dot org  2005-06-16 08:32 -------
Honza, this was introduced with your patch
http://gcc.gnu.org/ml/gcc-cvs/2005-06/msg00584.html

Could you please have a look?


-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |hubicka at gcc dot gnu dot
                   |                            |org


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

[Bug tree-optimization/22088] [4.1 regression] ICE with -ftree-vectorize

[hubicka at ucw dot cz]

------- Additional Comments From hubicka at ucw dot cz  2005-06-16 08:36 -------
Subject: Re:  [4.1 regression] ICE with -ftree-vectorize

> 
> ------- Additional Comments From reichelt at gcc dot gnu dot org  2005-06-16 08:32 -------
> Honza, this was introduced with your patch
> http://gcc.gnu.org/ml/gcc-cvs/2005-06/msg00584.html
> 
> Could you please have a look?

Looks like initialization/finalization of original/copy tables is
missing somewhere.  I will check.

Honza
> 
> 
> -- 
>            What    |Removed                     |Added
> ----------------------------------------------------------------------------
>                  CC|                            |hubicka at gcc dot gnu dot
>                    |                            |org
> 
> 
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=22088
> 
> ------- You are receiving this mail because: -------
> You are on the CC list for the bug, or are watching someone who is.


-- 


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

[Bug tree-optimization/22088] [4.1 regression] ICE with -ftree-vectorize

[belyshev at depni dot sinp dot msu dot ru]

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|                            |1
      Known to fail|                            |4.1.0
      Known to work|                            |4.0.0 4.0.1
   Last reconfirmed|0000-00-00 00:00:00         |2005-06-17 10:07:46
               date|                            |
   Target Milestone|---                         |4.1.0


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

[Bug tree-optimization/22088] [4.1 regression] ICE with -ftree-vectorize

[dorit at il dot ibm dot com]

------- Additional Comments From dorit at il dot ibm dot com  2005-06-17 12:58 -------
> bug.c: In function 'foo':
> bug.c:4: internal compiler error: in get_bb_copy, at cfg.c:1078

a bunch of vectorizer testcases are failing with this error on 
i686-pc-linux-gnu and powerpc-darwin

-- 


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

[Bug tree-optimization/22088] [4.1 regression] ICE with -ftree-vectorize

[hubicka at ucw dot cz]

------- Additional Comments From hubicka at ucw dot cz  2005-06-17 13:53 -------
Subject: Re:  [4.1 regression] ICE with -ftree-vectorize

> 
> ------- Additional Comments From dorit at il dot ibm dot com  2005-06-17 12:58 -------
> > bug.c: In function 'foo':
> > bug.c:4: internal compiler error: in get_bb_copy, at cfg.c:1078
> 
> a bunch of vectorizer testcases are failing with this error on 
> i686-pc-linux-gnu and powerpc-darwin

THe attached patch should fix it.  I wonder why it didn't show up in my
i686 regtest originally?  (are the vectorizer testcases disabled by
default or something?)


Index: tree-vect-transform.c
===================================================================
RCS file: /cvs/gcc/gcc/gcc/tree-vect-transform.c,v
retrieving revision 2.25
diff -c -3 -p -r2.25 tree-vect-transform.c
*** tree-vect-transform.c	10 Jun 2005 14:51:44 -0000	2.25
--- tree-vect-transform.c	17 Jun 2005 13:13:52 -0000
*************** vect_transform_loop (loop_vec_info loop_
*** 2048,2053 ****
--- 2048,2054 ----
    if (vect_print_dump_info (REPORT_DETAILS, UNKNOWN_LOC))
      fprintf (vect_dump, "=== vec_transform_loop ===");
  
+   initialize_original_copy_tables ();
    
    /* Peel the loop if there are data refs with unknown alignment.
       Only one data ref with unknown store is allowed.  */
*************** vect_transform_loop (loop_vec_info loop_
*** 2139,2144 ****
--- 2140,2147 ----
       until all the loops have been transformed?  */
    update_ssa (TODO_update_ssa);
  
+   free_original_copy_tables ();
+ 
    if (vect_print_dump_info (REPORT_VECTORIZED_LOOPS, LOOP_LOC (loop_vinfo)))
      fprintf (vect_dump, "LOOP VECTORIZED.");
  }


-- 


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

[Bug tree-optimization/22088] [4.1 regression] ICE with -ftree-vectorize

[rth at gcc dot gnu dot org]

------- Additional Comments From rth at gcc dot gnu dot org  2005-06-17 22:11 -------
Fixed: http://gcc.gnu.org/ml/gcc-patches/2005-06/msg01535.html

Although I didn't see this pr before writing the patch, I like my patch
better because not all paths go through loop peeling.  Which means that
we'd do the init/free in cases that don't need it.

No, the vectorization tests are never disabled.  We *always* at least do
the compile tests.

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


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

Re: [Bug tree-optimization/22088] [4.1 regression] ICE with -ftree-vectorize

[Jan Hubicka]

> 
> ------- Additional Comments From rth at gcc dot gnu dot org  2005-06-17 22:11 -------
> Fixed: http://gcc.gnu.org/ml/gcc-patches/2005-06/msg01535.html
> 
> Although I didn't see this pr before writing the patch, I like my patch
> better because not all paths go through loop peeling.  Which means that
> we'd do the init/free in cases that don't need it.

Ah, thanks for fixing that!  I just sent patch for that too, but too
late..
> 
> No, the vectorization tests are never disabled.  We *always* at least do
> the compile tests.

Weird, my scripts must have some kind of leak then.  I have to dig into
logs...

Honza
> 
> -- 
>            What    |Removed                     |Added
> ----------------------------------------------------------------------------
>              Status|NEW                         |RESOLVED
>          Resolution|                            |FIXED
> 
> 
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=22088

[Bug tree-optimization/22088] [4.1 regression] ICE with -ftree-vectorize

[hubicka at ucw dot cz]

------- Additional Comments From hubicka at ucw dot cz  2005-06-17 23:31 -------
Subject: Re:  [4.1 regression] ICE with -ftree-vectorize

> 
> ------- Additional Comments From rth at gcc dot gnu dot org  2005-06-17 22:11 -------
> Fixed: http://gcc.gnu.org/ml/gcc-patches/2005-06/msg01535.html
> 
> Although I didn't see this pr before writing the patch, I like my patch
> better because not all paths go through loop peeling.  Which means that
> we'd do the init/free in cases that don't need it.

Ah, thanks for fixing that!  I just sent patch for that too, but too
late..
> 
> No, the vectorization tests are never disabled.  We *always* at least do
> the compile tests.

Weird, my scripts must have some kind of leak then.  I have to dig into
logs...

Honza
> 
> -- 
>            What    |Removed                     |Added
> ----------------------------------------------------------------------------
>              Status|NEW                         |RESOLVED
>          Resolution|                            |FIXED
> 
> 
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=22088


-- 


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