<div dir="ltr"><div>+1<br><br></div>Yours,<br>Andrey<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Feb 20, 2014 at 4:32 PM, Hal Finkel <span dir="ltr"><<a href="mailto:hfinkel@anl.gov" target="_blank">hfinkel@anl.gov</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">----- Original Message -----<br>
> From: "Chandler Carruth" <<a href="mailto:chandlerc@google.com">chandlerc@google.com</a>><br>
> To: "C. Bergström" <<a href="mailto:cbergstrom@pathscale.com">cbergstrom@pathscale.com</a>><br>
> Cc: <a href="mailto:reviews%2BD2841%2Bpublic%2B0d4390c49e3f0815@llvm-reviews.chandlerc.com">reviews+D2841+public+0d4390c49e3f0815@llvm-reviews.chandlerc.com</a>, "Douglas Gregor" <<a href="mailto:dgregor@apple.com">dgregor@apple.com</a>>, "Hal<br>

> Finkel" <<a href="mailto:hfinkel@anl.gov">hfinkel@anl.gov</a>>, "Dmitri Gribenko" <<a href="mailto:gribozavr@gmail.com">gribozavr@gmail.com</a>>, <a href="mailto:fraggamuffin@gmail.com">fraggamuffin@gmail.com</a>, "a bataev"<br>

> <<a href="mailto:a.bataev@hotmail.com">a.bataev@hotmail.com</a>>, "llvm cfe" <<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a>><br>
> Sent: Thursday, February 20, 2014 5:15:54 AM<br>
> Subject: Re: [PATCH] [OPENMP] Replace libgomp by libiomp5 in driver<br>
><br>
><br>
><br>
><br>
><br>
> On Thu, Feb 20, 2014 at 2:57 AM, "C. Bergström" <<br>
> <a href="mailto:cbergstrom@pathscale.com">cbergstrom@pathscale.com</a> > wrote:<br>
><br>
><br>
><br>
> On 02/20/14 05:43 PM, Chandler Carruth wrote:<br>
><br>
><br>
><br>
> - There has been no effort to make this library even work properly<br>
> with Clang as a host compiler. See the copious notes that only Clang<br>
> 3.3 is supported, and that not full featured.<br>
> - The build system is totally disjoint from LLVM's, in fact it is an<br>
> entirely custom Perl build system that is unlike anything in use by<br>
> the LLVM project.<br>
> We have some changes which should allow building with clang and also<br>
> cmake build files to resolve a couple of your issues. I planned to<br>
> create a pull request for Intel in the near future.<br>
><br>
><br>
> I so don't get this.<br>
><br>
><br>
> The project is no longer an Intel project. It is an LLVM project. The<br>
> development *must* take place as an LLVM project, or nothing else<br>
> will matter.<br>
><br>
><br>
> You could help formulate this process by submitting patches to the<br>
> LLVM project and committing them there. That *needs* to be the<br>
> upstream to make this a real, viable, open source effort.<br>
><br>
><br>
> This should help make QA'ing libiomp5 easier.<br>
><br>
><br>
><br>
> This, of course, will be awesome. I look forward to better<br>
> documentation as well to clarify how to QA it. And tests with which<br>
> to QA it.<br>
><br>
><br>
> ----------<br>
> While I agree with all your points - pragmatically there is probably<br>
> very few real world clang + OMP users and the risk/impact is low.<br>
><br>
><br>
> This is simply false. I don't know why you think this. People here<br>
> have been using -fopenmp since the day they could link their<br>
> binaries which called out to an omp runtime library call. Yes, their<br>
> pragmas don't do anything, and their parallelism is terrible, but<br>
> they really are critically relying on the ability to build, link,<br>
> and execute (single threaded) programs with -fopenmp. Regressing a<br>
> documented and readily available feature of Clang with no warning<br>
> and no prior release cycle where these pieces were QA'ed and<br>
> released is pretty... bad.<br>
><br>
><br>
> I'd +1 this change since libiomp5 aligns with those who are actually<br>
> working on the code, using it and reviewing it.<br>
> I've not seen how this aligns with Dmitri or Doug's usages and I<br>
> think they've done essentially all of the review. I know it doesn't<br>
> align with any of my users that are relying on -fopenmp not<br>
> producing a link error.<br>
<br>
</div></div>I agree that changing the default at this point is premature; there are users following trunk who rely on the existing behavior, and they'll see no benefit from the current change. On the other hand, we need the ability to link against our own runtime library in order to start adding CodeGen support, so we do need this change, it just does not need to be the default yet.<br>

<br>
Why don't we add a -fopenmp-lib=[gomp|iomp] or -fopenmp-type or flavor or whatever (I don't have a strong opinion about the naming), and base things off of that? In the future, we can change the default, and we can leave CodeGen support disabled when not targeting our own runtime.<br>

<span class="HOEnZb"><font color="#888888"><br>
 -Hal<br>
<br>
--<br>
Hal Finkel<br>
Assistant Computational Scientist<br>
Leadership Computing Facility<br>
Argonne National Laboratory<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</div></div></blockquote></div><br></div>