[PATCH] [OPENMP] Added option -fopenmp=libiomp5

Dmitri Gribenko gribozavr at gmail.com
Thu Feb 27 04:30:10 PST 2014



================
Comment at: lib/Driver/Tools.cpp:5237-5240
@@ -5236,4 +5236,6 @@
 
-  if (Args.hasArg(options::OPT_fopenmp))
+  if (Args.hasArg(options::OPT_fopenmp)) {
     // This is more complicated in gcc...
     CmdArgs.push_back("-lgomp");
+  } else if (const Arg *A = Args.getLastArg(options::OPT_fopenmp_EQ)) {
+    if (StringRef(A->getValue()) == "libiomp5")
----------------
Alexey Bataev wrote:
> Dmitri Gribenko wrote:
> > What happens if the user passes -fopenmp -fopenmp=libiomp5?
> Then only -fopenmp will work and the warning for -fopenmp=libiomp5 will be emitted. I thought that we have to be conservative with -fopenmp option.
OK, please add a test.



http://llvm-reviews.chandlerc.com/D2841



More information about the cfe-commits mailing list