[PATCH] D17170: [OPENMP] Codegen for distribute directive

[Alexey Bataev via cfe-commits]

ABataev added inline comments.

================
Comment at: lib/CodeGen/CGOpenMPRuntime.cpp:2126
@@ +2125,3 @@
+  OpenMPSchedType ScheduleNum = getRuntimeSchedule(ScheduleKind, Chunk != nullptr,
+                                                Ordered);
+  auto *UpdatedLocation = emitUpdateLocation(CGF, Loc, OMP_IDENT_KMPC);
----------------
Wrong indentation

================
Comment at: lib/CodeGen/CGOpenMPRuntime.cpp:2127
@@ +2126,3 @@
+                                                Ordered);
+  auto *UpdatedLocation = emitUpdateLocation(CGF, Loc, OMP_IDENT_KMPC);
+  auto *ThreadId = getThreadID(CGF, Loc);
----------------
Just 'emitUpdateLocation(CGF, Loc)' please

================
Comment at: lib/CodeGen/CGOpenMPRuntime.cpp:2141
@@ +2140,3 @@
+  OpenMPSchedType ScheduleNum = getRuntimeSchedule(SchedKind, Chunk != nullptr);
+  auto *UpdatedLocation = emitUpdateLocation(CGF, Loc, OMP_IDENT_KMPC);
+  auto *ThreadId = getThreadID(CGF, Loc);
----------------
Also, 'emitUpdateLocation(CGF, Loc)'


Repository:
  rL LLVM

http://reviews.llvm.org/D17170