[PATCH] D15936: Update code in buildCudaActions and BuildActions to latest idiom.

Artem Belevich via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 6 14:10:50 PST 2016


tra accepted this revision.
tra added a comment.
This revision is now accepted and ready to land.

LGTM


================
Comment at: lib/Driver/Driver.cpp:1300
@@ -1299,5 +1299,3 @@
   for (Arg *A : Args) {
-    if (A->getOption().matches(options::OPT_cuda_gpu_arch_EQ)) {
-      A->claim();
-      if (GpuArchNames.insert(A->getValue()).second)
-        GpuArchList.push_back(A->getValue());
+    if (!A->getOption().matches(options::OPT_cuda_gpu_arch_EQ)) {
+      continue;
----------------
style nit -- no need for {}


http://reviews.llvm.org/D15936





More information about the cfe-commits mailing list