[Bug middle-end/34856] New: ICE with some constant vectors

[Bug middle-end/34856] New: ICE with some constant vectors

[pinskia at gcc dot gnu dot org]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain, Size: 956 bytes --]

Testcase:
#undef __vector
#define __vector __attribute__((vector_size(16) ))
typedef __vector signed char qword;
typedef __vector unsigned int VU32;
extern short g[192 +16];
void f(qword);
void f1 (unsigned ctr)
{
  VU32 pin;
  pin = (VU32){(unsigned int)&g[16]};
  do {
   f((qword)pin);
   ctr--;
  }
while(ctr);
}

---- CUT ----
The ICE at -O1 and above:
t.c: In function ‘calc_tentexps_opm’:
t.c:13: error: invalid reference prefix
{(unsigned int) &tentwork[16]}


-- 
           Summary: ICE with some constant vectors
           Product: gcc
           Version: 4.3.0
            Status: UNCONFIRMED
          Keywords: ice-on-valid-code
          Severity: normal
          Priority: P3
         Component: middle-end
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: pinskia at gcc dot gnu dot org
GCC target triplet: spu-elf, powerpc-linux-gnu


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

[Bug middle-end/34856] ICE with some constant vectors

[ismail at pardus dot org dot tr]

------- Comment #1 from ismail at pardus dot org dot tr  2008-01-18 19:45 -------
on i686 linux I get;

test.c:16: internal compiler error: in for_each_index, at
tree-ssa-loop-im.c:222

works with gcc 3.4.6.


-- 


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

[Bug middle-end/34856] [4.3 Regression] ICE with some constant vectors

[pinskia at gcc dot gnu dot org]

-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|ICE with some constant      |[4.3 Regression] ICE with
                   |vectors                     |some constant vectors
   Target Milestone|---                         |4.3.0


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

[Bug middle-end/34856] [4.3 Regression] ICE with some constant vectors

[rguenth at gcc dot gnu dot org]

-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1
           Priority|P3                          |P2
   Last reconfirmed|0000-00-00 00:00:00         |2008-01-18 20:44:51
               date|                            |


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

[Bug middle-end/34856] [4.3 Regression] ICE with some constant vectors

[dominiq at lps dot ens dot fr]

------- Comment #2 from dominiq at lps dot ens dot fr  2008-01-18 20:52 -------
On i686-apple-darwin9, I get:

pr34856.c: In function 'f1':
pr34856.c:16: error: invalid reference prefix
{(unsigned int) &g[16]}

pr34856.c:16: internal compiler error: verify_stmts failed


-- 


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

[Bug middle-end/34856] [4.3 Regression] ICE with some constant vectors

[rguenth at gcc dot gnu dot org]

------- Comment #3 from rguenth at gcc dot gnu dot org  2008-01-21 13:42 -------
Confirmed.  In the latter case we const-propped the initializer into the
V_C_E:

VIEW_CONVERT_EXPR<vector signed char>({(unsigned int) &g[16]})

which is of course neither CONSTANT_CLASS_P nor a gimple lvalue (but it is
is_gimple_min_invariant, which is why we propagate it in the first place):

    /* Vector constant constructors are gimple invariant.  */
    case CONSTRUCTOR:
      if (TREE_TYPE (t) && TREE_CODE (TREE_TYPE (t)) == VECTOR_TYPE)
        return TREE_CONSTANT (t);
      else
        return false;

The point is, that we do allow V_C_E expressions on rvalues, so the
verification is simply bogus.


-- 


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

[Bug middle-end/34856] [4.3 Regression] ICE with some constant vectors

[rguenth at gcc dot gnu dot org]

------- Comment #4 from rguenth at gcc dot gnu dot org  2008-01-21 13:54 -------
And for_each_index does not handle CONSTRUCTOR.  Then, we ICE in 
copy_reference_ops_from_ref, at tree-ssa-sccvn.c:574
because that obviously also does not handle CONSTRUCTOR either.

_Then_, at -O2, we ICE in simplify_immed_subreg (x86_64 -m32 -msse)

#1  0x000000000085ed6b in simplify_immed_subreg (outermode=V16QImode, 
    op=0x2ae82bd7b6b0, innermode=V4SImode, byte=0)
    at /space/rguenther/src/svn/trunk/gcc/simplify-rtx.c:4761
4761              gcc_unreachable ();
(gdb) call debug_rtx (op)
(const_vector:V4SI [
        (const:SI (plus:SI (symbol_ref:SI ("g") [flags 0x40] <var_decl
0x2ae82c6450a0 g>)
                (const_int 32 [0x20])))
        (const_int 0 [0x0])
        (const_int 0 [0x0])
        (const_int 0 [0x0])
    ])

as we don't handle a const:SI wrapped first vector element.  I'm going to
"fix" the non-rtl cases.


-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |rguenth at gcc dot gnu dot
                   |dot org                     |org
             Status|NEW                         |ASSIGNED
   Last reconfirmed|2008-01-18 20:44:51         |2008-01-21 13:54:34
               date|                            |


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

[Bug middle-end/34856] [4.3 Regression] ICE with some constant vectors

[rguenth at gcc dot gnu dot org]

------- Comment #5 from rguenth at gcc dot gnu dot org  2008-01-21 16:22 -------
Subject: Bug 34856

Author: rguenth
Date: Mon Jan 21 16:21:45 2008
New Revision: 131696

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=131696
Log:
2008-01-21  Richard Guenther  <rguenther@suse.de>

        PR middle-end/34856
        * tree-cfg.c (verify_expr): Allow all invariant expressions
        instead of just constant class ones as reference argument.
        * tree-ssa-loop-im.c (for_each_index): Handle CONSTRUCTOR
        like any other constant.
        * tree-ssa-sccvn.c (copy_reference_ops_from_ref): Likewise.

        * gcc.c-torture/compile/pr34856.c: New testcase.

Added:
    trunk/gcc/testsuite/gcc.c-torture/compile/pr34856.c
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/testsuite/ChangeLog
    trunk/gcc/tree-cfg.c
    trunk/gcc/tree-ssa-loop-im.c
    trunk/gcc/tree-ssa-sccvn.c


-- 


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

[Bug rtl-optimization/34856] [4.3 Regression] ICE with some constant vectors

[rguenth at gcc dot gnu dot org]

------- Comment #6 from rguenth at gcc dot gnu dot org  2008-01-21 16:24 -------
tree parts fixed.  Eric, can you look into the RTL parts?


-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |ebotcazou at gcc dot gnu dot
                   |                            |org
         AssignedTo|rguenth at gcc dot gnu dot  |unassigned at gcc dot gnu
                   |org                         |dot org
             Status|ASSIGNED                    |NEW
          Component|middle-end                  |rtl-optimization


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

[Bug rtl-optimization/34856] [4.3 Regression] ICE with some constant vectors

[dominiq at lps dot ens dot fr]

------- Comment #8 from dominiq at lps dot ens dot fr  2008-01-22 23:50 -------
On i686-apple-darwin9, testsuite/gcc.c-torture/compile/pr34856.c gives an ICE
at -O3 with trunk rev.131745:

[ibook-dhum] f90/bug% /opt/gcc/w1-gcc4.3w/bin/gcc -O3
/opt/gcc/_gcc_clean/gcc/testsuite/gcc.c-torture/compile/pr34856.c
/opt/gcc/_gcc_clean/gcc/testsuite/gcc.c-torture/compile/pr34856.c: In function
'f1':
/opt/gcc/_gcc_clean/gcc/testsuite/gcc.c-torture/compile/pr34856.c:16: error:
invalid reference prefix
{(unsigned int) &g[16]}

/opt/gcc/_gcc_clean/gcc/testsuite/gcc.c-torture/compile/pr34856.c:16: internal
compiler error: verify_stmts failed


-- 


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

[Bug middle-end/34856] [4.3 Regression] ICE with some constant vectors

[pinskia at gcc dot gnu dot org]

------- Comment #9 from pinskia at gcc dot gnu dot org  2008-01-23 00:31 -------
(In reply to comment #4)
> _Then_, at -O2, we ICE in simplify_immed_subreg (x86_64 -m32 -msse)

I get this same ICE for spu-elf too.


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
          Component|rtl-optimization            |middle-end
           Priority|P2                          |P3
   Target Milestone|4.3.0                       |---


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

[Bug middle-end/34856] [4.3 Regression] ICE with some constant vectors

[pinskia at gcc dot gnu dot org]

-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |4.3.0


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

[Bug middle-end/34856] [4.3 Regression] ICE with some constant vectors

[rguenther at suse dot de]

------- Comment #10 from rguenther at suse dot de  2008-01-23 09:21 -------
Subject: Re:  [4.3 Regression] ICE with some
 constant vectors

On Tue, 22 Jan 2008, dominiq at lps dot ens dot fr wrote:

> ------- Comment #8 from dominiq at lps dot ens dot fr  2008-01-22 23:50 -------
> On i686-apple-darwin9, testsuite/gcc.c-torture/compile/pr34856.c gives an ICE
> at -O3 with trunk rev.131745:
> 
> [ibook-dhum] f90/bug% /opt/gcc/w1-gcc4.3w/bin/gcc -O3
> /opt/gcc/_gcc_clean/gcc/testsuite/gcc.c-torture/compile/pr34856.c
> /opt/gcc/_gcc_clean/gcc/testsuite/gcc.c-torture/compile/pr34856.c: In function
> 'f1':
> /opt/gcc/_gcc_clean/gcc/testsuite/gcc.c-torture/compile/pr34856.c:16: error:
> invalid reference prefix
> {(unsigned int) &g[16]}
> 
> /opt/gcc/_gcc_clean/gcc/testsuite/gcc.c-torture/compile/pr34856.c:16: internal
> compiler error: verify_stmts failed

Works for me.  The tree-cfg.c hunk was supposed to fix this.  Instead
it should ICE in

internal compiler error: in simplify_immed_subreg, at simplify-rtx.c:4761

Richard.


-- 


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

[Bug middle-end/34856] [4.3 Regression] ICE with some constant vectors

[dominiq at lps dot ens dot fr]

------- Comment #11 from dominiq at lps dot ens dot fr  2008-01-23 09:28 -------
> The tree-cfg.c hunk was supposed to fix this.

Additional info: on i686-apple-darwin9 the failures disappear with -m64.


-- 


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

[Bug middle-end/34856] [4.2/4.3 Regression] ICE with some constant vectors

[rguenth at gcc dot gnu dot org]

------- Comment #12 from rguenth at gcc dot gnu dot org  2008-01-23 22:35 -------
The ICE in simplify_immed_subreg is reproducible with 4.2.3.


-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Known to fail|                            |4.2.3 4.3.0
      Known to work|                            |4.1.2
           Priority|P3                          |P2
            Summary|[4.3 Regression] ICE with   |[4.2/4.3 Regression] ICE
                   |some constant vectors       |with some constant vectors


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

[Bug middle-end/34856] [4.2/4.3 Regression] ICE with some constant vectors

[pinskia at gcc dot gnu dot org]

------- Comment #13 from pinskia at gcc dot gnu dot org  2008-01-23 22:40 -------
Most likely we should just return NULL here instead of crashing.


-- 


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

[Bug middle-end/34856] [4.2/4.3 Regression] ICE with some constant vectors

[ebotcazou at gcc dot gnu dot org]

------- Comment #14 from ebotcazou at gcc dot gnu dot org  2008-01-24 11:29 -------
This is invalid in the first place:

(const_vector:V4SI [
   (const:SI (plus:SI (symbol_ref:SI ("g") [flags 0x40] <var_decl 0x556e70b0
g>)
                (const_int 32 [0x20])))
   (const_int 0 [0x0])
   (const_int 0 [0x0])
   (const_int 0 [0x0])

CONST_VECTOR can only contain CONST_INTs or CONST_DOUBLEs (and CONST_FIXED ?).

ix86_expand_vector_init (as well as rs6000_expand_vector_init) should be fixed
to avoid generating them.


-- 

ebotcazou at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|ebotcazou at gcc dot gnu dot|unassigned at gcc dot gnu
                   |org                         |dot org
             Status|ASSIGNED                    |NEW


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

[Bug middle-end/34856] [4.2/4.3 Regression] ICE with some constant vectors

[pinskia at gcc dot gnu dot org]

------- Comment #15 from pinskia at gcc dot gnu dot org  2008-01-24 11:35 -------
(In reply to comment #14)
> ix86_expand_vector_init (as well as rs6000_expand_vector_init) should be fixed
> to avoid generating them.
I guess then I will handle spu_expand_vector_init then but it wouldn't be until
this weekend.


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 GCC target triplet|spu-elf, powerpc-linux-gnu  |spu-elf, powerpc*-*-*, i*68-
                   |                            |*-*, x86_64-*-*


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

[Bug middle-end/34856] [4.2/4.3 Regression] ICE with some constant vectors

[ebotcazou at gcc dot gnu dot org]

------- Comment #16 from ebotcazou at gcc dot gnu dot org  2008-01-24 11:48 -------
> I guess then I will handle spu_expand_vector_init then but it wouldn't be
> until this weekend.

Btw, is this **** code real-life code?  If no, why is this P2?


-- 


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

[Bug middle-end/34856] [4.2/4.3 Regression] ICE with some constant vectors

[pinskia at gcc dot gnu dot org]

------- Comment #17 from pinskia at gcc dot gnu dot org  2008-01-24 12:12 -------
(In reply to comment #16)
> Btw, is this **** code real-life code?  If no, why is this P2?

It comes originally from a PS3 SDK sources, SPU code in fact.


-- 


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

[Bug middle-end/34856] [4.2/4.3 Regression] ICE with some constant vectors

[pinskia at gcc dot gnu dot org]

------- Comment #18 from pinskia at gcc dot gnu dot org  2008-01-24 12:16 -------
I should say this was reduced from the PS3 SDK sources.  
The   
pin = (VU32){(unsigned int)&g[16]}; 

line exists in the original source.

I am totally serious this is what the code looks like.


-- 


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

[Bug target/34856] [4.2/4.3 Regression] ICE with some constant vectors

[rguenth at gcc dot gnu dot org]

------- Comment #19 from rguenth at gcc dot gnu dot org  2008-01-24 12:17 -------
P2 as all ICE-on-valid code for primary and secondary targets are at least P2.
Uros, can you handle x86?


-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |uros at gcc dot gnu dot org
          Component|middle-end                  |target


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

[Bug target/34856] [4.2/4.3 Regression] ICE with some constant vectors

[ubizjak at gmail dot com]

------- Comment #20 from ubizjak at gmail dot com  2008-01-24 13:48 -------
(In reply to comment #19)
> Uros, can you handle x86?

Sometimes CONSTANT_P is just too broad to construct correct RTX. This patch
creates most sensible asm (i.e. using movd instead of movdqa):

--cut here--
Index: i386.c
===================================================================
--- i386.c      (revision 131776)
+++ i386.c      (working copy)
@@ -23593,7 +23593,9 @@ ix86_expand_vector_init (bool mmx_ok, rt
   for (i = 0; i < n_elts; ++i)
     {
       x = XVECEXP (vals, 0, i);
-      if (!CONSTANT_P (x))
+      if (!(CONST_INT_P (x)
+           || GET_CODE (x) == CONST_DOUBLE
+           || GET_CODE (x) == CONST_FIXED))
        n_var++, one_var = i;
       else if (x != CONST0_RTX (inner_mode))
        all_const_zero = false;
--cut here--

f1:
        pushl   %ebx
        subl    $24, %esp
        movl    32(%esp), %ebx
        movd    .LC0, %xmm0
        movdqa  %xmm0, (%esp)
        .p2align 4,,7
        .p2align 3
.L2:
        movdqa  (%esp), %xmm0
        call    f
        subl    $1, %ebx
        jne     .L2
        addl    $24, %esp
        popl    %ebx
        ret
        .size   f1, .-f1
        .section        .rodata.cst4,"aM",@progbits,4
        .align 4
.LC0:
        .long   g+32


-- 


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

[Bug target/34856] [4.2/4.3 Regression] ICE with some constant vectors

[dominiq at lps dot ens dot fr]

------- Comment #21 from dominiq at lps dot ens dot fr  2008-01-24 14:57 -------
Prliminary tests show that the patch in #20 fix the problem on
i686-apple-darwin9, regtesting.


-- 


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

[Bug target/34856] [4.2/4.3 Regression] ICE with some constant vectors

[dominiq at lps dot ens dot fr]

------- Comment #22 from dominiq at lps dot ens dot fr  2008-01-24 16:33 -------
I get the following summary for the gcc regression testsuite (32/64 bit modes)
with the patch in #20:

                === gcc Summary for unix/-m64 ===

# of expected passes            47421
# of unexpected failures        11
# of unexpected successes       1
# of expected failures          163
# of untested testcases         35
# of unsupported tests          477

                === gcc Summary ===

# of expected passes            94971
# of unexpected failures        23
# of unexpected successes       3
# of expected failures          326
# of unresolved testcases       1
# of untested testcases         70
# of unsupported tests          843
/opt/gcc/i686-darwin/gcc/xgcc  version 4.3.0 20080124 (experimental) (GCC) 

For comparison without the patch, see pr34483 comment #53.


-- 


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

[Bug target/34856] [4.2/4.3 Regression] ICE with some constant vectors

[uros at gcc dot gnu dot org]

------- Comment #23 from uros at gcc dot gnu dot org  2008-01-24 17:09 -------
Subject: Bug 34856

Author: uros
Date: Thu Jan 24 17:08:10 2008
New Revision: 131796

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=131796
Log:
        PR target/34856
        * config/i386/i386.c (ix86_expand_vector_init): Consider only
        CONST_INT, CONST_DOUBLE and CONST_FIXED as constant vector elements.

testsuite/ChangeLog:

        PR target/34856
        * gcc.dg/pr34856.c: New test.


Added:
    trunk/gcc/testsuite/gcc.dg/pr34856.c
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/config/i386/i386.c
    trunk/gcc/testsuite/ChangeLog


-- 


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

[Bug target/34856] [4.2/4.3 Regression] ICE with some constant vectors

[ubizjak at gmail dot com]

------- Comment #24 from ubizjak at gmail dot com  2008-01-24 17:12 -------
x86 part is fixed by http://gcc.gnu.org/ml/gcc-patches/2008-01/msg01146.html.

This patch also added a testcase that will ATM fail for RS6000 and SPU.


-- 

ubizjak at gmail dot com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 GCC target triplet|spu-elf, powerpc*-*-*, i*68-|spu-elf, powerpc*-*-*
                   |*-*, x86_64-*-*             |


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

[Bug target/34856] [4.2/4.3 Regression] ICE with some constant vectors

[uros at gcc dot gnu dot org]

------- Comment #25 from uros at gcc dot gnu dot org  2008-01-25 10:26 -------
Subject: Bug 34856

Author: uros
Date: Fri Jan 25 10:25:21 2008
New Revision: 131821

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=131821
Log:
        PR target/34856
        * simplifx-rtx.c (simplify_const_binary_operation) [VEC_CONCAT]:
        Consider only CONST_INT, CONST_DOUBLE and CONST_FIXED as constant
        vector elements.

testsuite/ChangeLog:

        PR target/34856
        * gcc.dg/pr34856.c: Use pointer mode for "pin".


Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/simplify-rtx.c
    trunk/gcc/testsuite/ChangeLog
    trunk/gcc/testsuite/gcc.dg/pr34856.c


-- 


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

[Bug target/34856] [4.2/4.3 Regression] ICE with some constant vectors

[ubizjak at gmail dot com]

------- Comment #26 from ubizjak at gmail dot com  2008-01-28 12:04 -------
*** Bug 34995 has been marked as a duplicate of this bug. ***


-- 

ubizjak at gmail dot com changed:

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


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

[Bug target/34856] [4.2/4.3/4.4 Regression] ICE with some constant vectors

[rguenth at gcc dot gnu dot org]

-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|4.3.0                       |4.2.4


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

[Bug target/34856] [4.2/4.3/4.4 Regression] ICE with some constant vectors

[jsm28 at gcc dot gnu dot org]

------- Comment #27 from jsm28 at gcc dot gnu dot org  2008-05-19 20:24 -------
4.2.4 is being released, changing milestones to 4.2.5.


-- 

jsm28 at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|4.2.4                       |4.2.5


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

[Bug target/34856] [4.2/4.3/4.4 Regression] ICE with some constant vectors

[uweigand at gcc dot gnu dot org]

------- Comment #28 from uweigand at gcc dot gnu dot org  2008-06-28 10:48 -------
Subject: Bug 34856

Author: uweigand
Date: Sat Jun 28 10:47:36 2008
New Revision: 137218

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=137218
Log:
        PR target/34856
        * config/spu/spu.c (spu_builtin_splats): Do not generate
        invalid CONST_VECTOR expressions.
        (spu_expand_vector_init): Likewise.

Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/config/spu/spu.c


-- 


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

[Bug target/34856] [4.2/4.3/4.4 Regression] ICE with some constant vectors

[uweigand at gcc dot gnu dot org]

------- Comment #29 from uweigand at gcc dot gnu dot org  2008-06-28 10:49 -------
Subject: Bug 34856

Author: uweigand
Date: Sat Jun 28 10:48:33 2008
New Revision: 137219

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=137219
Log:
        PR target/34856
        * config/spu/spu.c (spu_builtin_splats): Do not generate
        invalid CONST_VECTOR expressions.
        (spu_expand_vector_init): Likewise.

Modified:
    branches/gcc-4_3-branch/gcc/ChangeLog
    branches/gcc-4_3-branch/gcc/config/spu/spu.c


-- 


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

[Bug target/34856] [4.2/4.3 Regression] ICE with some constant vectors

[pinskia at gcc dot gnu dot org]

------- Comment #30 from pinskia at gcc dot gnu dot org  2008-08-16 23:31 -------
This was fixed for PPC between:
-LAST_UPDATED: Mon Jun 23 19:12:32 UTC 2008 (revision 137049)
+LAST_UPDATED: Tue Jul  8 16:22:03 UTC 2008 (revision 137638)

I don't have any closer date though.  So this is no longer a 4.4 regression for
PPC. For spu, both 4.3 and 4.4 were fixed so removing that from the target
triple.


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 GCC target triplet|spu-elf, powerpc*-*-*       |powerpc*-*-*
      Known to work|4.1.2                       |4.1.2 4.4.0
            Summary|[4.2/4.3/4.4 Regression] ICE|[4.2/4.3 Regression] ICE
                   |with some constant vectors  |with some constant vectors


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

[Bug target/34856] [4.2/4.3 Regression] ICE with some constant vectors

[luisgpm at linux dot vnet dot ibm dot com]

------- Comment #31 from luisgpm at linux dot vnet dot ibm dot com  2008-09-09 13:51 -------
I have the fix for PPC. Any special reason why this doesn't get reproduced
there? Still would be worthwhile to include the rs6000-specific fix for this
bug ticket?

Thanks,
Luis


-- 


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

[Bug target/34856] [4.3 Regression] ICE with some constant vectors

[jsm28 at gcc dot gnu dot org]

------- Comment #32 from jsm28 at gcc dot gnu dot org  2009-03-31 20:17 -------
Closing 4.2 branch.


-- 

jsm28 at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|[4.2/4.3 Regression] ICE    |[4.3 Regression] ICE with
                   |with some constant vectors  |some constant vectors
   Target Milestone|4.2.5                       |4.3.4


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

[Bug target/34856] [4.3 Regression] ICE with some constant vectors

[rguenth at gcc dot gnu dot org]

------- Comment #33 from rguenth at gcc dot gnu dot org  2009-08-04 12:28 -------
GCC 4.3.4 is being released, adjusting target milestone.


-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|4.3.4                       |4.3.5


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

[Bug target/34856] [4.3 Regression] ICE with some constant vectors

[rguenth at gcc dot gnu dot org]

------- Comment #34 from rguenth at gcc dot gnu dot org  2010-05-22 18:11 -------
GCC 4.3.5 is being released, adjusting target milestone.


-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|4.3.5                       |4.3.6


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

[Bug target/34856] [4.3 Regression] ICE with some constant vectors

[rguenth at gcc dot gnu.org]

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

Richard Guenther <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
      Known to work|                            |
         Resolution|                            |FIXED
   Target Milestone|4.3.6                       |4.4.0
      Known to fail|                            |

--- Comment #35 from Richard Guenther <rguenth at gcc dot gnu.org> 2011-06-27 11:17:49 UTC ---
Fixed for GCC 4.4.0.