[Bug fortran/29779] [4.3 Regression] vectorizer fortran testcases failing
--- 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
--- 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 <[EMAIL PROTECTED]> 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
[Bug fortran/29779] [4.3 Regression] vectorizer fortran testcases failing
--- 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
--- 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
--- 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
--- 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
--- 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
--- 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
--- 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
--- 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|-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
--- 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
--- 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
--- 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 <[EMAIL PROTECTED]> * 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
--- 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 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
