[PATCH] [OPENMP] Initial codegen for '#pragma omp parallel'

[hfinkel at anl.gov]

================
Comment at: lib/CodeGen/CGOpenMPRuntime.h:48
@@ +47,3 @@
+    OMP_IDENT_IMD = 0x01,
+    /// \brief Use c-stile ident structure.
+    OMP_IDENT_KMPC = 0x02,
----------------
c-style?

================
Comment at: lib/CodeGen/CGOpenMPRuntime.h:45
@@ +44,3 @@
+public:
+  enum OpenMPLocationFlags {
+    /// \brief Use tramponline for internal microtask.
----------------
This should have some overall comment explaining what this is the location of.

================
Comment at: lib/CodeGen/CGOpenMPRuntime.h:72
@@ +71,3 @@
+private:
+  enum Ident_TFields {
+    Ident_T_Reserved_1,
----------------
This needs to be documented (including the fields too).

================
Comment at: lib/CodeGen/CGOpenMPRuntime.h:80
@@ +79,3 @@
+  CodeGenModule &CGM;
+  llvm::Constant *DefaultOpenMPPSource;
+  llvm::StructType *Ident_TTy;
----------------
What's this?

================
Comment at: lib/CodeGen/CGOpenMPRuntime.cpp:28
@@ +27,3 @@
+CGOpenMPRuntime::CGOpenMPRuntime(CodeGenModule &CGM) : CGM(CGM) {
+  DefaultOpenMPPSource = CGM.GetAddrOfConstantCString(";unknown;unknown;0;0;;");
+  DefaultOpenMPPSource =
----------------
Please document the meaning of this string.

================
Comment at: lib/Sema/SemaOpenMP.cpp:675
@@ +674,3 @@
+        { std::make_pair("global_tid", KmpInt32PtrTy),
+          std::make_pair("bound_tid", KmpInt32PtrTy),
+          std::make_pair(StringRef(), QualType()) // __context with shared vars
----------------
Could these names conflict with user-provided names? (Should they be in the implementation namespace?)


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