[PATCH] D52259: [CUDA] Rearrange search path ordering to fix two test case failures

Jiading Gai via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Sep 18 21:46:59 PDT 2018


gaijiading created this revision.
gaijiading added reviewers: jlebar, Hahnfeld, dlj, tra.
gaijiading added a project: clang.
Herald added a subscriber: cfe-commits.

Before this patch, when a system has the CUDA toolkit already installed to its default locations in /usr/local/cuda, there will be two unexpected test case failures:

1. <clang.src>/test/Driver/cuda-detect.cu
2. <clang.src>/test/Driver/cuda-macosx.cu

The reason being that the overall search order was set up to always pick up the path with well-known tools such as ptxas in it; However, the above two test cases require the CUDA path to be the one that is passed to --sysroot.

Moving the sysroot path on top fixes the issue.


Repository:
  rC Clang

https://reviews.llvm.org/D52259

Files:
  unittests/Analysis/ExprMutationAnalyzerTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D52259.166066.patch
Type: text/x-patch
Size: 61659 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20180919/8bd9210d/attachment-0001.bin>


More information about the cfe-commits mailing list