Hahnfeld added inline comments.

================
Comment at: lib/Driver/ToolChains/Cuda.cpp:665
 
+    // Add path to lib / lib64 folder.
+    SmallString<256> DefaultLibPath =
----------------
ABataev wrote:
> You're changing the order of the lookup for the paths. Is this intended? If 
> so, you need the test for this.
Yes, that's explained in the summary.

I'll try to see if there is a way to test this, even if we can't expect 
libomptarget-nvptx to be built together with Clang. At the moment I think only 
`LIBRARY_PATH` is tested, I'm adding some for the new 
`--libomptarget-nvptx-path`.


Repository:
  rC Clang

https://reviews.llvm.org/D51686



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

Reply via email to