[Openmp-commits] [PATCH] D96187: [OpenMP] Fix libomptarget tests to find libomp

[Joel E. Denny via Phabricator via Openmp-commits]

jdenny marked an inline comment as done.
jdenny added inline comments.


================
Comment at: openmp/libomptarget/deviceRTLs/nvptx/test/lit.site.cfg.in:8
 config.binary_dir = "@CMAKE_CURRENT_BINARY_DIR@"
+config.libomp_library_dir = "@LIBOMP_LIBRARY_DIR@"
 config.library_dir = "@LIBOMPTARGET_LIBRARY_DIR@"
----------------
tianshilei1992 wrote:
> We probably don't need an extra variable for that. `config.omp_host_rtl_directory` is for `libomp` if I'm not mistaken.
Yep, that's better.  Thanks.


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

https://reviews.llvm.org/D96187