tra accepted this revision.
tra added inline comments.
This revision is now accepted and ready to land.


================
Comment at: clang/lib/Driver/ToolChains.cpp:1819
         Args.getLastArgValue(options::OPT_cuda_path_EQ));
-  else {
+  else if (HostTriple.isOSLinux() || HostTriple.isMacOSX()) {
     CudaPathCandidates.push_back(D.SysRoot + "/usr/local/cuda");
----------------
I'd keep unix-like paths as defaults under else {}.
If you do want to enumerate supported OS variants explicitly, then me want to 
throw an error if particular host OS is not supported.



https://reviews.llvm.org/D28320



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

Reply via email to