[PATCH] D23042: [CUDA] Do not allow using NVPTX target for host compilation.

Justin Lebar via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 2 13:31:26 PDT 2016


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

> Restore assertions for presence of -march flag.


We don't need an explicit assertion in TranslateArgs?


================
Comment at: lib/Driver/Driver.cpp:1412
@@ +1411,3 @@
+  assert(HostTC && "No toolchain for host compilation.");
+  if (HostTC->getTriple().isNVPTX()) {
+    C.getDriver().Diag(diag::err_drv_cuda_nvptx_host);
----------------
This is probably worth a comment?


https://reviews.llvm.org/D23042





More information about the cfe-commits mailing list