This is an archive of the discontinued LLVM Phabricator instance.

[test-suite, CUDA] Fix some CMake problems
ClosedPublic

Authored by Hahnfeld on Aug 25 2018, 6:52 AM.

Details

Summary
  1. The regex was matching any number in the path which is a problem if your path to external/ had numbers in it. I think this was because of the wrong string escape.
  2. If /thrust does not exist, the variable isn't set at all. Avoid failure on EXIST with empty path by using DEFINED.

Diff Detail

Repository
rL LLVM

Event Timeline

Hahnfeld created this revision.Aug 25 2018, 6:52 AM
tra accepted this revision.Aug 27 2018, 11:34 AM
This revision is now accepted and ready to land.Aug 27 2018, 11:34 AM
This revision was automatically updated to reflect the committed changes.