[Openmp-commits] [PATCH] D95155: [libomptarget] Build cuda plugin without cuda installed locally

Jon Chesterfield via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Fri Jan 22 09:28:31 PST 2021


JonChesterfield marked 2 inline comments as done.
JonChesterfield added inline comments.


================
Comment at: openmp/libomptarget/plugins/cuda/dynamic_cuda/cuda.cpp:63
+
+  const char *CudaLib = "libcuda.so";
+  void *dynlib_handle = dlopen(CudaLib, RTLD_NOW);
----------------
jdoerfert wrote:
> Should we make this a global, or a CMAKE variable?
Some user control over this is probably good, e.g. people might want to load a specific libcuda with several installed, and passing a path would allow that.

Simple approach is probably a macro that can be passed in from cmake to override the default


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D95155/new/

https://reviews.llvm.org/D95155



More information about the Openmp-commits mailing list