[cfe-commits] OpenMP-Patch-1: fopenmp option support

Mahesha HS mahesha.llvm at gmail.com
Tue Oct 30 09:50:20 PDT 2012


Ok. Then, let me implement it now in this way. If required we the
change logic in future.

--
mahesha

On Tue, Oct 30, 2012 at 10:16 PM, Dmitri Gribenko <gribozavr at gmail.com> wrote:
> On Tue, Oct 30, 2012 at 6:32 AM, Mahesha HS <mahesha.llvm at gmail.com> wrote:
>> Hi Hal and Ron,
>>
>> Here is possibilities, for -fopenmp and -fno-openmp combinations. Is
>> it looks okay?
>>
>> 1. Neither -fopenmp nor -fno-openmp is passed.
>>        if (Translation unit has OpenMP source)
>>             Do emit warning.
>>        else
>>             Do nothing.
>>
>> 2. Only -fopenmp is passed
>>          if (Translation unit has OpenMP source)
>>              Do process it.
>>          else
>>              Do nothing.
>>
>> 3. Only -fno-openmp is passed
>>          if (Translation unit has OpenMP source)
>>              Do nothing
>>          else
>>              Do nothing
>>
>> Note: It is actually no-op.
>
> Not really, it silenced warnings!
>
> In general: this seems to agree with how other -fno-foo options work.
>
> 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 at gmail.com>*/



-- 
mahesha



More information about the cfe-commits mailing list