[Openmp-commits] [PATCH] D107074: [libomptarget][nfc] Only set cuda-path for nvptx tests

[Jon Chesterfield via Phabricator via Openmp-commits]

JonChesterfield created this revision.
JonChesterfield added reviewers: protze.joachim, tianshilei1992, jhuber6, ye-luo.
Herald added a subscriber: yaxunl.
JonChesterfield requested review of this revision.
Herald added a project: OpenMP.
Herald added a subscriber: openmp-commits.

Remove --cuda-path=CUDA_TOOLKIT_ROOT_DIR-NOTFOUND
from the invocation of non-nvptx test cases. Better signal
to noise ratio on other architectures.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D107074

Files:
  openmp/libomptarget/test/lit.cfg


Index: openmp/libomptarget/test/lit.cfg
===================================================================
--- openmp/libomptarget/test/lit.cfg
+++ openmp/libomptarget/test/lit.cfg
@@ -197,9 +197,9 @@
 config.substitutions.append(("%clangxx", config.test_cxx_compiler))
 config.substitutions.append(("%clang", config.test_c_compiler))
 config.substitutions.append(("%openmp_flags", config.test_openmp_flags))
-if config.cuda_path:
-  config.substitutions.append(("%cuda_flags", "--cuda-path=" + config.cuda_path))
+if config.libomptarget_current_target.startswith('nvptx') and config.cuda_path:
+    config.substitutions.append(("%cuda_flags", "--cuda-path=" + config.cuda_path))
 else:
-  config.substitutions.append(("%cuda_flags", ""))
+    config.substitutions.append(("%cuda_flags", ""))
 config.substitutions.append(("%flags", config.test_flags))
 config.substitutions.append(("%not", config.libomptarget_not))


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D107074.362768.patch
Type: text/x-patch
Size: 918 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/openmp-commits/attachments/20210729/43292b79/attachment.bin>