----- Original Message -----
> From: "Dmitri Gribenko" <[email protected]>
> To: "Hal Finkel" <[email protected]>
> Cc: "Chandler Carruth" <[email protected]>, "Douglas Gregor"
> <[email protected]>, "benny kra"
> <[email protected]>, "Alexey Bataev" <[email protected]>, "llvm cfe"
> <[email protected]>
> Sent: Monday, December 17, 2012 11:14:07 AM
> Subject: Re: [cfe-commits] [PATCH] First OpenMP patch
>
> On Mon, Dec 17, 2012 at 6:26 PM, Hal Finkel <[email protected]> wrote:
> > ----- Original Message -----
> >> From: "Chandler Carruth" <[email protected]>
> >> To: "Hal Finkel" <[email protected]>
> >> Cc: "Dmitri Gribenko" <[email protected]>, "benny kra"
> >> <[email protected]>, "Alexey Bataev" <[email protected]>,
> >> "llvm cfe" <[email protected]>
> >> Sent: Monday, December 17, 2012 8:42:03 AM
> >> Subject: Re: [cfe-commits] [PATCH] First OpenMP patch
> >>
> >>
> >>
> >> On Mon, Dec 17, 2012 at 5:45 AM, Hal Finkel < [email protected] >
> >> wrote:
> >>
> >>
> >>
> >>
> >>
> >> ----- Original Message -----
> >> > From: "Dmitri Gribenko" < [email protected] >
> >> > To: "Alexey Bataev" < [email protected] >
> >> > Cc: "Hal Finkel" < [email protected] >, [email protected] ,
> >> > "mahesha llvm" < [email protected] >, "benny kra"
> >> > < [email protected] >
> >> > Sent: Monday, December 17, 2012 6:45:07 AM
> >> > Subject: Re: [cfe-commits] [PATCH] First OpenMP patch
> >> >
> >>
> >> > On Mon, Dec 17, 2012 at 11:48 AM, Alexey Bataev <
> >> > [email protected]
> >> > >
> >> > wrote:
> >> > > Dmitry, Hal,
> >> > > Thank you for your comments and sorry for the delay, I was on
> >> > > a
> >> > > vacation.
> >> > > I've made some fixes according to your comments.
> >> > > Hal, I've change the sentence in the doc. Now there is only
> >> > > one
> >> > > warning, if
> >> > > any OpenMP pragma is found.
> >> > > Dmitry, I've changed the processing of the -fno-openmp flag.
> >> > > If
> >> > > -fno-openmp
> >> > > is specified, the option -Wno-source-uses-openmp is passed to
> >> > > the
> >> > > frontend.
> >> > > Option -Wsource-uses-openmp is on by default.
> >> >
> >> > Hello Alexey,
> >> >
> >> > This patch looks good to me.
> >>
> >> This also looks good to me. In nobody objects in the next day or
> >> so,
> >> please commit.
> >>
> >>
> >> Sorry that this got lost Hal, but I have said on another thread
> >> about
> >> this patch (but with a different author) that I don't really think
> >> we should add documentation and the beginnings of support for
> >> -fopen-mp without first having a clear discussion and document
> >> describing the expected design of OpenMP support in Clang.
> >
> > Regarding the documentation, I agree. It would be better to leave
> > it out at this point (we probably don't want docs for such an
> > incomplete feature to appear on the web site at present), although
> > it should still be developed.
>
> If this is a concern (it is in my opinion), just put a ".. warning::"
> on top of the page saying that OpenMP support is under development.
>
> Keeping documentation in the repository is much better since anyone
> touching OpenMP can be directed to the in-repository document (to
> read
> or update it), instead of pointing somewhere to the outside.
Agreed. Even so, we may want not to link to it from elsewhere.
>
> What is more important than documentation, is the change in behavior.
> Someone might have used '-fopenmp' with Clang before, and now they
> will get an incomplete implementation. We might also introduce a
> warning to Clang like "experimental OpenMP support enabled" for the
> time being.
Good idea.
-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 <[email protected]>*/
>
--
Hal Finkel
Postdoctoral Appointee
Leadership Computing Facility
Argonne National Laboratory
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
