<span style='font-family:Verdana'><span style='font-size:12px'>Dmitry, Hal,<br /> 
Thank you for your comments and sorry for the delay, I was on a 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 warning, if any OpenMP pragma is found.<br /> 
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.<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: Dmitri Gribenko</span></span></p> 
        <p style="margin:0px; padding:0px;" > 
                <span style="font-family:Verdana"><span style="font-size:12px">Sent: 12/12/12 12:39 PM</span></span></p> 
        <p style="margin:0px; padding:0px;" > 
                <span style="font-family:Verdana"><span style="font-size:12px">To: Alexey Bataev</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> 
                        <pre style="white-space: pre-wrap; word-wrap: break-word; font-size:11;pre"> 
On Wed, Dec 12, 2012 at 6:10 AM, Alexey Bataev <a.bataev@gmx.com> wrote: 
> Hello guys, 
> Are there any comments on this version of patch? If no, then can I commit? 

Sorry for a long delay.  I do have comments. 

+// Only -Wsource-uses-openmp is passed. 
+// RUN: %clang -### -S -Wsource-uses-openmp -o - %s 2>&1 | FileCheck 
--check-prefix=CHECK-WOMP %s 
+// CHECK-WOMP: -Wsource-uses-openmp 
+// RUN: %clang -### -S -Wsource-uses-openmp -o - %s 2>&1 | FileCheck 
--check-prefix=CHECK-WOMPN %s 
+// CHECK-WOMPN-NOT: -Wno-source-uses-openmp 

It doesn't make sense to test this by itself.  This aspect of the 
Clang warning infrastructure is working well and is tested elsewhere. 

Sorry if that was unclear, but what I meant to ask to test is: if 
-Wsource-uses-openmp is passed or not depending on whether 
-fopenmp/-fno-openmp is passed. 

+  // -fno-openmp switch processing 
+  if (Args.hasArg(OPT_fno_openmp)) { 
+    Opts.Warnings.push_back("no-source-uses-openmp"); 
+  } 

Now I see that since warning is controlled from the frontend, rather 
than the driver, it is not possible to test the logic with -###.  I 
wonder if it is possible (and makes sense!) to move this logic to the 
driver, since the frontend should not make such decisions. 

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@gmail.com>*/</pre> 
                </div> 
        </div> 
</blockquote> 
<p style="margin:0px; padding:0px;" > 
         </p> 
<br /> 
</span></span>