<div style="font-family: arial, helvetica, sans-serif; font-size: 10pt"><div dir="ltr">On Mon, Dec 17, 2012 at 5:45 AM, Hal Finkel <span dir="ltr"><<a href="mailto:hfinkel@anl.gov" target="_blank" class="cremed">hfinkel@anl.gov</a>></span> wrote:<br>
<div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">----- Original Message -----<br>
> From: "Dmitri Gribenko" <<a href="mailto:gribozavr@gmail.com" class="cremed">gribozavr@gmail.com</a>><br>
> To: "Alexey Bataev" <<a href="mailto:a.bataev@gmx.com" class="cremed">a.bataev@gmx.com</a>><br>
> Cc: "Hal Finkel" <<a href="mailto:hfinkel@anl.gov" class="cremed">hfinkel@anl.gov</a>>, <a href="mailto:cfe-commits@cs.uiuc.edu" class="cremed">cfe-commits@cs.uiuc.edu</a>, "mahesha llvm" <<a href="mailto:mahesha.llvm@gmail.com" class="cremed">mahesha.llvm@gmail.com</a>>, "benny kra"<br>

> <<a href="mailto:benny.kra@gmail.com" class="cremed">benny.kra@gmail.com</a>><br>
> Sent: Monday, December 17, 2012 6:45:07 AM<br>
> Subject: Re: [cfe-commits] [PATCH] First OpenMP patch<br>
><br>
</div><div class="im">> On Mon, Dec 17, 2012 at 11:48 AM, Alexey Bataev <<a href="mailto:a.bataev@gmx.com" class="cremed">a.bataev@gmx.com</a>><br>
> wrote:<br>
> > Dmitry, Hal,<br>
> > Thank you for your comments and sorry for the delay, I was on a<br>
> > vacation.<br>
> > I've made some fixes according to your comments.<br>
> > Hal, I've change the sentence in the doc. Now there is only one<br>
> > warning, if<br>
> > any OpenMP pragma is found.<br>
> > Dmitry, I've changed the processing of the -fno-openmp flag. If<br>
> > -fno-openmp<br>
> > is specified, the option -Wno-source-uses-openmp is passed to the<br>
> > frontend.<br>
> > Option -Wsource-uses-openmp is on by default.<br>
><br>
> Hello Alexey,<br>
><br>
> This patch looks good to me.<br>
<br>
</div>This also looks good to me. In nobody objects in the next day or so, please commit.</blockquote><div><br></div><div style>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.</div>
<div style><br></div><div style>Essentially, I think this patch is starting ast step 2, 3, or 4 rather than step 1.</div><div style><br></div><div style>I think we're actually imagining OpenMP working generally the same way these days (based on the discussion we had on the aforementioned thread -- another reason I really dislike forking threads), but I'd like to get that documented, and a general roadmap of how the support is going to be added to Clang and by whom laid out first. And I think based on *that* discussion, Doug needs to sign off on OpenMP as a viable language extension for Clang to support. I think it is viable, and would support it, but it's not obvious under the current rules. Does this make sense to folks? I think it's the next step.</div>
<div style><br></div><div style><br></div><div style>Finally, I would point out that I still have some concerns over the contributors of OpenMP support not being significant contributors to Clang in general, and not contributing to the support and maintenance burden of the system as a whole. I have not yet seen significant changes there, although I'm hopeful they'll be forthcoming. I think that is an essential component to getting these patches in and adding OpenMP to the Clang.</div>
</div></div></div></div>