[PATCH] [OPENMP] -fopenmp enables OpenMP support (fix for http://llvm.org/PR23492)

[Richard Smith]

================
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/