sfantao added inline comments.

================
Comment at: lib/CodeGen/CodeGenModule.cpp:218
@@ -204,1 +217,3 @@
+  }
+  return;
 }
----------------
ABataev wrote:
> I think this must be removed
Ok, done.

================
Comment at: lib/Frontend/CompilerInvocation.cpp:1837
@@ +1836,3 @@
+      
Diags.Report(clang::diag::err_drv_omp_host_or_device_target_not_supported)
+          << TargetOpts.Triple << Opts.OpenMPIsDevice;
+      break;
----------------
ABataev wrote:
> OpenMPIsDevice is always false here. Do we really need it here?
Ok, hardcoded it to false.


http://reviews.llvm.org/D16784



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to