[Bug middle-end/29779] New: vectorizer fortran testcases failing

[Bug middle-end/29779] New: vectorizer fortran testcases failing

[dorit at il dot ibm dot com]

Looks like sometime between Oct27 
(http://gcc.gnu.org/ml/gcc-testresults/2006-10/msg01336.html) 
and Oct30 
(http://gcc.gnu.org/ml/gcc-testresults/2006-10/msg01538.html) 
the fortran vectorizer testcases started ICEing on:

gfortran.dg/vect/vect-3.f90:0: warning: 'const' attribute directive ignored
gfortran.dg/vect/vect-3.f90:4: internal compiler error: in
vect_setup_realignment, at tree-vect-transform.c:2534

Should be related somehow to this code in rs6000.c:

      /* Initialize target builtin that implements
         targetm.vectorize.builtin_mask_for_load.  */

      decl = add_builtin_function ("__builtin_altivec_mask_for_load",
                                   v16qi_ftype_long_pcvoid,
                                   ALTIVEC_BUILTIN_MASK_FOR_LOAD,
                                   BUILT_IN_MD, NULL,
                                   tree_cons (get_identifier ("const"),
                                              NULL_TREE, NULL_TREE));


Anybody knows which patch caused this?


-- 
           Summary: vectorizer fortran testcases failing
           Product: gcc
           Version: 4.3.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: middle-end
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: dorit at il dot ibm dot com
 GCC build triplet: ppc*-*-linux
  GCC host triplet: ppc*-*-linux
GCC target triplet: ppc*-*-linux


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

[Bug fortran/29779] [4.3 Regression] vectorizer fortran testcases failing

[pinskia at gcc dot gnu dot org]

-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |diagnostic, ice-on-valid-
                   |                            |code
            Summary|vectorizer fortran testcases|[4.3 Regression] vectorizer
                   |failing                     |fortran testcases failing
   Target Milestone|---                         |4.3.0


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

[Bug fortran/29779] [4.3 Regression] vectorizer fortran testcases failing

[pault at gcc dot gnu dot org]

------- Comment #1 from pault at gcc dot gnu dot org  2006-11-09 20:04 -------
Dorit,

Nothing stands out in the gfortran patches in that interval, although I am not
sure what I am looking for.  I base my remark on the fact that none of the
patches between 10/27 and 10/31 would appear to touch that subroutine.  The
code generated for vect-3.f90 looks OK, so I would have to suppose that the
problem has arisen further downstream.

Regards

Paul


-- 


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

[Bug fortran/29779] [4.3 Regression] vectorizer fortran testcases failing

[pinskia at gcc dot gnu dot org]

------- Comment #2 from pinskia at gcc dot gnu dot org  2006-11-11 00:32 -------
Actually I think this bug was caused by:
2006-10-23 Rafael Avila de Espindola  <rafael.espindola@gmail.com>

        * tree.c (local_define_builtin): Replace calls to
        lang_hooks.builtin_function with add_builtin_function.
        * c-objc-common.h (LANG_HOOKS_BUILTIN_FUNCTION): Define as
        c_builtin_function.
        * c-tree.h (builtin_function): Rename to c_builtin_function. Change
        the signature.
        * c-decl.c (builtin_function): Rename to c_builtin_function. Move
        common code to add_builtin_function.
...


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  GCC build triplet|ppc*-*-linux                |
   GCC host triplet|ppc*-*-linux                |
 GCC target triplet|ppc*-*-linux                |powerpc*-*-*


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

[Bug fortran/29779] [4.3 Regression] vectorizer fortran testcases failing

[pinskia at gcc dot gnu dot org]

------- Comment #3 from pinskia at gcc dot gnu dot org  2006-11-11 00:34 -------
*** Bug 29617 has been marked as a duplicate of this bug. ***


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |edmar at freescale dot com


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

[Bug fortran/29779] [4.3 Regression] vectorizer fortran testcases failing

[pinskia at gcc dot gnu dot org]

------- Comment #4 from pinskia at gcc dot gnu dot org  2006-11-11 00:36 -------
I am going to look into this, this weekend and see why that patch caused a
difference.


-- 


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

[Bug fortran/29779] [4.3 Regression] vectorizer fortran testcases failing

[pinskia at gcc dot gnu dot org]

------- Comment #5 from pinskia at gcc dot gnu dot org  2006-11-12 08:20 -------
I am going to try to fix this, it blocks my other work on getting altivec
builtins marked as const/pure.


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         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         |2006-11-12 08:20:20
               date|                            |


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

[Bug fortran/29779] [4.3 Regression] vectorizer fortran testcases failing

[pinskia at gcc dot gnu dot org]

------- Comment #6 from pinskia at gcc dot gnu dot org  2006-11-12 19:58 -------
This patch should fix the problem:
Index: config/rs6000/rs6000.c
===================================================================
--- config/rs6000/rs6000.c      (revision 118728)
+++ config/rs6000/rs6000.c      (working copy)
@@ -8711,9 +8711,8 @@ altivec_init_builtins (void)
       decl = add_builtin_function ("__builtin_altivec_mask_for_load",
                                   v16qi_ftype_long_pcvoid,
                                   ALTIVEC_BUILTIN_MASK_FOR_LOAD,
-                                  BUILT_IN_MD, NULL,
-                                  tree_cons (get_identifier ("const"),
-                                             NULL_TREE, NULL_TREE));
+                                  BUILT_IN_MD, NULL, NULL);
+      TREE_READONLY (decl) = 1;
       /* Record the decl. Will be used by rs6000_builtin_mask_for_load.  */
       altivec_builtin_mask_for_load = decl;
     }

I don't think we should be using attributes here really.


-- 


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

[Bug fortran/29779] [4.3 Regression] vectorizer fortran testcases failing

[dorit at il dot ibm dot com]

------- Comment #7 from dorit at il dot ibm dot com  2006-11-17 06:46 -------
(In reply to comment #6)
> This patch should fix the problem:

indeed it does, thanks!
are you going to submit it to mainline?


-- 


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

[Bug fortran/29779] [4.3 Regression] vectorizer fortran testcases failing

[pinskia at gcc dot gnu dot org]

------- Comment #8 from pinskia at gcc dot gnu dot org  2006-11-17 15:11 -------
(In reply to comment #7)
> indeed it does, thanks!
> are you going to submit it to mainline?

Should be submitted this weekend, today is a party day for me, launch and all
:).


-- 


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

[Bug fortran/29779] [4.3 Regression] vectorizer fortran testcases failing

[burnus at gcc dot gnu dot org]

------- Comment #9 from burnus at gcc dot gnu dot org  2006-12-05 20:51 -------
I think this was fixed by:
http://gcc.gnu.org/ml/gcc-cvs/2006-11/msg00427.html


-- 

burnus at gcc dot gnu dot org changed:

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


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

[Bug fortran/29779] [4.3 Regression] vectorizer fortran testcases failing

[pinskia at gcc dot gnu dot org]

------- Comment #10 from pinskia at gcc dot gnu dot org  2006-12-05 22:15 -------
(In reply to comment #9)
> I think this was fixed by:
> http://gcc.gnu.org/ml/gcc-cvs/2006-11/msg00427.html

No it was not.


-- 

pinskia at gcc dot gnu dot org changed:

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


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

[Bug fortran/29779] [4.3 Regression] vectorizer fortran testcases failing

[burnus at gcc dot gnu dot org]

------- Comment #11 from burnus at gcc dot gnu dot org  2006-12-06 22:10 -------
> > I think this was fixed by:
> > http://gcc.gnu.org/ml/gcc-cvs/2006-11/msg00427.html
> No it was not.
Thanks for checking. (This was actually the PR, for which I asked yesterday on
IRC whether it was commited or not.)

By the way, you wrote 2006-11-17:
> Should be submitted this weekend
Any new ETA?


-- 


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

[Bug fortran/29779] [4.3 Regression] vectorizer fortran testcases failing

[dorit at il dot ibm dot com]

------- Comment #12 from dorit at il dot ibm dot com  2006-12-06 22:22 -------

> By the way, you wrote 2006-11-17:
> > Should be submitted this weekend
> Any new ETA?

It was already submitted: 
http://gcc.gnu.org/ml/gcc-patches/2006-12/msg00110.html


-- 


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

[Bug fortran/29779] [4.3 Regression] vectorizer fortran testcases failing

[pinskia at gcc dot gnu dot org]

------- Comment #14 from pinskia at gcc dot gnu dot org  2006-12-19 08:29 -------
Fixed.


-- 

pinskia at gcc dot gnu dot org changed:

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


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

[Bug fortran/29779] [4.3 Regression] vectorizer fortran testcases failing

[pinskia at gcc dot gnu dot org]

------- Comment #13 from pinskia at gcc dot gnu dot org  2006-12-19 08:28 -------
Subject: Bug 29779

Author: pinskia
Date: Tue Dec 19 08:28:46 2006
New Revision: 120045

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=120045
Log:
2006-12-18  Andrew Pinski  <pinskia@gmail.com>

        PR target/29779
        * config/rs6000/rs6000.c (altivec_init_builtins): Change to set
        READONLY on __builtin_altivec_mask_for_load instead of adding
        the attribute.



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


-- 


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