[PATCH] D18171: [CUDA][OpenMP] Create generic offload action

[Samuel Antao via cfe-commits]

sfantao added inline comments.

================
Comment at: lib/Driver/Action.cpp:191-202
@@ +190,14 @@
+  auto I = getInputs().begin();
+  auto E = getInputs().end();
+  if (I == E)
+    return;
+
+  // Skip host action
+  if (HostTC)
+    ++I;
+
+  auto TI = DevToolChains.begin();
+  for (; I != E; ++I, ++TI) {
+    assert(TI != DevToolChains.end() && "More dependences than tool chains??");
+    Work(*I, *TI, (*I)->getOffloadingArch());
+  }
----------------
The only change in this function was the assertion inside the loop. Its possible my message got in before the actual diff, sorry about that...

Ok, got it. I'm replacing the assertion in the loop body by an assertion that checks if the sizes of the inputs  and toolchains are consistent. Let me know if you'd rather have a check of the iterator inside and after the loop.


http://reviews.llvm.org/D18171