> -----Original Message----- > From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- > ow...@gcc.gnu.org] On Behalf Of Aldy Hernandez > Sent: Wednesday, May 01, 2013 12:02 PM > To: Iyer, Balaji V > Cc: Jakub Jelinek; Richard Henderson; gcc-patches@gcc.gnu.org > Subject: Re: [gomp4] Some progress on #pragma omp simd > > On 05/01/13 10:58, Iyer, Balaji V wrote: > >> > >> Well, it looks like things are bit simpler than expected. > >> > >> Multiple vectorlength clauses are being deprecated or eliminated in > >> the upcoming spec. So it looks like vectorlength is the same thing > >> as the safelen clause. > >> > >> If you agree then I can get rid of OMP_CLAUSE_CILK_VECTORLENGTH and > >> just emit an OMP_CLAUSE_SAFELEN. > >> > >> Agreed? > > > > To my best knowledge, Yes. I believe safelen requires/allows only 1 > > value, so we should do what Jakub mentioned (vectorlength (a, b, c)) > > should be converted to safelen (max(a,b,c)) > > Well, we don't even need to do max(a,b,c) because according to the Cilk Plus > forum, the upcoming spec does not allow multiple vectorlength clauses (or > arguments), so only vectorlength(const_expr) is allowed. > > Did I misunderstand something here?
I am not sure about the upcoming spec, but as per today's spec vectorlength in pragma simd can have multiple parameters and so I think it would be a good idea to use a max. Thanks, Balaji V. Iyer.
