This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: fix c/17384


On Mon, 18 Oct 2004 07:29:15 -0400, Andrew Pinski
<pinskia@physics.uc.edu> wrote:
> 
> On Oct 18, 2004, at 4:36 AM, Richard Guenther wrote:
> 
> > On Sun, 17 Oct 2004 16:06:13 -0700, Richard Henderson <rth@redhat.com>
> > wrote:
> >> On Sun, Oct 17, 2004 at 11:18:03PM +0200, Richard Guenther wrote:
> >>> I'm now getting
> >>> glsimd.h: warning: specifying vector types with __attribute__
> >>> ((mode))
> >>> is deprecated
> >>> glsimd.h: warning: use __attribute__ ((vector_size)) instead
> >>> glsimd.h: error: mode 'V4SF' applied to inappropriate type
> >>>
> >>> for
> >>>
> >>> typedef float v4sf __attribute__((mode(V4SF)));
> >>
> >> For mainline?
> >
> > Yes, for mainline as of yesterday.
> >
> >> Have you considered following the directions and using vector_size?
> >
> > But then it won't compile with 3.3 anymore (but I didn't try).  I
> > guess you maybe broke support for __attribute__((mode()))) with your
> > patch?
> 
> vector_size has been supported since 3.1 IIRC, it was added for altivec
> support.

But even the 3.4.2 manual still talks about __attribute__((mode)) in sec. 5.43.
I guess a patch from mainline needs to be backported, if the desired attribute
is vector_size rather than mode for vector types.  To 3.3, too, I presume.

Richard.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]