[cfe-commits] r169909 - in /cfe/trunk: include/clang/Driver/Options.td lib/Driver/Tools.cpp test/Driver/clang_f_opts.c

[Hal Finkel]

----- Original Message -----
> From: "Dmitri Gribenko" <gribozavr at gmail.com>
> To: "Hal Finkel" <hfinkel at anl.gov>
> Cc: cfe-commits at cs.uiuc.edu
> Sent: Tuesday, December 11, 2012 3:00:37 PM
> Subject: Re: [cfe-commits] r169909 - in /cfe/trunk: include/clang/Driver/Options.td lib/Driver/Tools.cpp
> test/Driver/clang_f_opts.c
> 
> On Tue, Dec 11, 2012 at 9:59 PM, Hal Finkel <hfinkel at anl.gov> wrote:
> > Author: hfinkel
> > Date: Tue Dec 11 13:59:32 2012
> > New Revision: 169909
> >
> > URL: http://llvm.org/viewvc/llvm-project?rev=169909&view=rev
> > Log:
> > Add -fslp-vectorize to enable bb-vectorize
> >
> > Add -fslp-vectorize (with -ftree-slp-vectorize as an alias for gcc
> > compatibility)
> > to provide a way to enable the basic-block vectorization pass. This
> > uses the same
> > acronym as gcc, superword-level parallelism (SLP), also common in
> > the literature,
> > to refer to basic-block vectorization.
> 
> Hi Hal,
> 
> Does the attached patch (adding SLP to the acronym list) look good?

Yes, although I would write Basic-Block Vectorization (with a dash).

Thanks again,
Hal

> 
> Dmitri
> 
> --
> main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if
> (j){printf("%d\n",i);}}} /*Dmitri Gribenko <gribozavr at gmail.com>*/
> 

-- 
Hal Finkel
Postdoctoral Appointee
Leadership Computing Facility
Argonne National Laboratory