[Openmp-commits] [PATCH] D89974: [driver][CUDA] Use CMake's FindCUDA as default --cuda-path.

[Artem Belevich via Phabricator via Openmp-commits]

tra added a comment.

In D89974#2348181 <https://reviews.llvm.org/D89974#2348181>, @Hahnfeld wrote:

> Don't we already have this via `PATH`? At least that was my motivation back then and it worked without problems.

We do and it should indeed work for the tests. Setting an environment variable should be reasonably low burden if/when default path is not good enough.

PATH does suffer from the problem that ptxas location is not necessarily the right CUDA location. I.e. sometimes it's found via /usr/bin which I can not drop from the PATH, so one should be careful to *prepend* desired ptxas location.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89974