<span style='font-family:Verdana'><span style='font-size:12px'>Hello everyone,<br />
Thank you very much for comments. I agree, that we should prepare some design description document. I'll try to prepapre it as soon as possible.<br />
I nthe meantime, I've prepared a new patch which emits warning when -fopenmp option is found and changed doc file from html to rst.<br />
Best regards,<br />
Alexey Bataev<br />
<br />
<p style="margin:0px; padding:0px;" >
</p>
<blockquote style="border-left: 1px solid #CCC; padding-left: 5px; margin-left: 5px; margin-bottom: 0px; margin-top: 0px; margin-right: 0px;" type="cite">
<p style="margin:0px; padding:0px;" >
<span style="font-family:Verdana"><span style="font-size:12px">----- Original Message -----</span></span></p>
<p style="margin:0px; padding:0px;" >
<span style="font-family:Verdana"><span style="font-size:12px">From: Chandler Carruth</span></span></p>
<p style="margin:0px; padding:0px;" >
<span style="font-family:Verdana"><span style="font-size:12px">Sent: 12/17/12 10:31 PM</span></span></p>
<p style="margin:0px; padding:0px;" >
<span style="font-family:Verdana"><span style="font-size:12px">To: Dmitri Gribenko</span></span></p>
<p style="margin:0px; padding:0px;" >
<span style="font-family:Verdana"><span style="font-size:12px">Subject: Re: [cfe-commits] [PATCH] First OpenMP patch</span></span></p>
<br />
<div>
<div style="font-family: arial, helvetica, sans-serif; font-size: 10pt">
<div>
On Mon, Dec 17, 2012 at 9:14 AM, Dmitri Gribenko <span><<a class="cremed" href="mailto:gribozavr@gmail.com" target="_blank">gribozavr@gmail.com</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="HOEnZb">
<div class="h5">
What is more important than documentation, is the change in behavior.</div>
</div>
Someone might have used '-fopenmp' with Clang before, and now they<br />
will get an incomplete implementation. We might also introduce a<br />
warning to Clang like "experimental OpenMP support enabled" for the<br />
time being.</blockquote>
<div>
</div>
<div style="">
Just to be clear, this is what I was worried about.</div>
<div style="">
</div>
<div style="">
Previously, the '-fopenmp' flag was handled just like many GCC flags that LLVM will accept and ignore. Now there appears to be support for the feature, but it doesn't do what the user wants...</div>
<div style="">
</div>
<div style="">
Anyways, the real point is that we should have some write up (preferably committed) which describes the plan and rationale for OpenMP support in Clang. Otherwise we're likely te stay in the exact spot you point out Hal: FIXME's all over the place for unimplemented functionality. =/ That's actually what I'm hoping to avoid.</div>
</div>
</div>
</div>
</div>
</div>
</blockquote>
<p style="margin:0px; padding:0px;" >
</p>
<br />
</span></span>