================
Comment at: lib/Driver/Tools.cpp:3812-3813
@@ +3811,4 @@
+  // Forward flags for OpenMP
+  Args.AddLastArg(CmdArgs, options::OPT_fopenmp);
+  Args.AddLastArg(CmdArgs, options::OPT_fopenmp_EQ);
+
----------------
Is there a reason to forward both of these to the frontend? It seems more 
inline with the design goal of giving the frontend a minimal and explicit 
interface to either rewrite `-fopenmp` to `-fopenmp=libiomp5` in the driver or 
to not expose the underlying library to the frontend at all (it doesn't seem 
like the frontend needs to care).

http://reviews.llvm.org/D9736

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/



_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to