[PATCH] D28320: [Driver] Driver changes to support CUDA compilation on Windows.

[Artem Belevich via Phabricator via cfe-commits]

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