[Bug c++/35758] [4.3/4.4 Regression] vector_size attribute lost in function arguments for templates

2008-04-24 Thread jakub at gcc dot gnu dot org
--- Comment #17 from jakub at gcc dot gnu dot org 2008-04-24 16:30 --- Subject: Bug 35758 Author: jakub Date: Thu Apr 24 16:29:40 2008 New Revision: 134639 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=134639 Log: PR c++/35758 * c-common.c

[Bug c++/35758] [4.3/4.4 Regression] vector_size attribute lost in function arguments for templates

2008-04-24 Thread jakub at gcc dot gnu dot org
--- Comment #18 from jakub at gcc dot gnu dot org 2008-04-24 16:32 --- Subject: Bug 35758 Author: jakub Date: Thu Apr 24 16:31:59 2008 New Revision: 134640 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=134640 Log: PR c++/35758 * c-common.c

[Bug c++/35758] [4.3/4.4 Regression] vector_size attribute lost in function arguments for templates

2008-04-22 Thread jakub at gcc dot gnu dot org
--- Comment #16 from jakub at gcc dot gnu dot org 2008-04-22 09:48 --- Especially for 4.3.x getting rid of the HACK. GROSS. This is absolutely disgusting. might not be something desirable for a stable branch. We've lived with this gross absolutely disgusting hack for more than 7 years,

[Bug c++/35758] [4.3/4.4 Regression] vector_size attribute lost in function arguments for templates

2008-04-07 Thread jakub at gcc dot gnu dot org
--- Comment #15 from jakub at gcc dot gnu dot org 2008-04-07 10:52 --- I've tried the first step - putting the attributes that require type to TYPE_ATTRIBUTES rather than DECL_ATTRIBUTES, see below. Unfortunately tsubst doesn't call apply_late_template_attributes in that case (the only

[Bug c++/35758] [4.3/4.4 Regression] vector_size attribute lost in function arguments for templates

2008-04-04 Thread jason at redhat dot com
--- Comment #12 from jason at redhat dot com 2008-04-04 18:10 --- Subject: Re: [4.3/4.4 Regression] vector_size attribute lost in function arguments for templates jakub at gcc dot gnu dot org wrote: Actually, to clarify #c10, attributes on parameter packs just make things harder on

[Bug c++/35758] [4.3/4.4 Regression] vector_size attribute lost in function arguments for templates

2008-04-04 Thread pinskia at gcc dot gnu dot org
--- Comment #13 from pinskia at gcc dot gnu dot org 2008-04-04 18:18 --- We also need to make sure the target specific attribute __altivec__ (powerPC) and __spu_vector__ (SPU) works correctly just as vector_size works. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=35758

[Bug c++/35758] [4.3/4.4 Regression] vector_size attribute lost in function arguments for templates

2008-04-04 Thread pinskia at gcc dot gnu dot org
--- Comment #14 from pinskia at gcc dot gnu dot org 2008-04-04 19:02 --- Hmm, for some reason __spu_vector__ works correctly for my testcase but not __altivec__; I have not looked into why though. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=35758

[Bug c++/35758] [4.3/4.4 Regression] vector_size attribute lost in function arguments for templates

2008-04-03 Thread jakub at gcc dot gnu dot org
--- Comment #10 from jakub at gcc dot gnu dot org 2008-04-03 10:23 --- See http://gcc.gnu.org/ml/gcc-patches/2008-04/msg00177.html for details. I guess before moving further along with this, attributes for parameter packs should be decided upon. Do we want to allow them at all? What

[Bug c++/35758] [4.3/4.4 Regression] vector_size attribute lost in function arguments for templates

2008-04-03 Thread jakub at gcc dot gnu dot org
--- Comment #11 from jakub at gcc dot gnu dot org 2008-04-03 11:28 --- Actually, to clarify #c10, attributes on parameter packs just make things harder on the compiler side, but even in C++98 the same issue is present: #define vector __attribute__((__vector_size__ (16))) template

[Bug c++/35758] [4.3/4.4 Regression] vector_size attribute lost in function arguments for templates

2008-04-01 Thread jakub at gcc dot gnu dot org
--- Comment #9 from jakub at gcc dot gnu dot org 2008-04-01 13:31 --- Testing a patch with a langhook. -- jakub at gcc dot gnu dot org changed: What|Removed |Added

[Bug c++/35758] [4.3/4.4 Regression] vector_size attribute lost in function arguments for templates

2008-03-31 Thread jakub at gcc dot gnu dot org
--- Comment #8 from jakub at gcc dot gnu dot org 2008-03-31 11:11 --- I think this shows that vector_size attribute can't be a late template attribute whenever processing_template_decl, it can be only a late template attribute if the decl is actually type or value dependent. So I

[Bug c++/35758] [4.3/4.4 Regression] vector_size attribute lost in function arguments for templates

2008-03-30 Thread pinskia at gcc dot gnu dot org
--- Comment #6 from pinskia at gcc dot gnu dot org 2008-03-31 00:16 --- So what is happening is we are not applying the attribute while doing overload resolution so we reject the template function then. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=35758

[Bug c++/35758] [4.3/4.4 Regression] vector_size attribute lost in function arguments for templates

2008-03-30 Thread pinskia at gcc dot gnu dot org
--- Comment #7 from pinskia at gcc dot gnu dot org 2008-03-31 00:18 --- That being said, I think fn_type_unification should apply the attributes but I don't know/understand the template part of the C++ front-end. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=35758

[Bug c++/35758] [4.3/4.4 Regression] vector_size attribute lost in function arguments for templates

2008-03-29 Thread pinskia at gcc dot gnu dot org
--- Comment #1 from pinskia at gcc dot gnu dot org 2008-03-29 18:45 --- This is caused by the delaying of vector_size attribute inside templates. -- pinskia at gcc dot gnu dot org changed: What|Removed |Added

[Bug c++/35758] [4.3/4.4 Regression] vector_size attribute lost in function arguments for templates

2008-03-29 Thread pinskia at gcc dot gnu dot org
--- Comment #2 from pinskia at gcc dot gnu dot org 2008-03-29 18:46 --- This blocks me from even thinking about updating the PS3 toolchain to 4.3.0. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=35758

[Bug c++/35758] [4.3/4.4 Regression] vector_size attribute lost in function arguments for templates

2008-03-29 Thread pinskia at gcc dot gnu dot org
--- Comment #3 from pinskia at gcc dot gnu dot org 2008-03-29 18:47 --- This is related to PR 27433 and I bet fixing this one will also fix that one. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=35758

[Bug c++/35758] [4.3/4.4 Regression] vector_size attribute lost in function arguments for templates

2008-03-29 Thread pinskia at gcc dot gnu dot org
--- Comment #4 from pinskia at gcc dot gnu dot org 2008-03-29 18:49 --- It worked with 4.3.0 20070623 and 4.2.0 20061019. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=35758

[Bug c++/35758] [4.3/4.4 Regression] vector_size attribute lost in function arguments for templates

2008-03-29 Thread rguenth at gcc dot gnu dot org
--- Comment #5 from rguenth at gcc dot gnu dot org 2008-03-29 18:55 --- Confirmed. -- rguenth at gcc dot gnu dot org changed: What|Removed |Added