Re: r248433 - [CUDA] Added CUDA installation detector class.

2015-09-23 Thread Artem Belevich via cfe-commits
Interesting. So it's sysroot option that's somehow to blame. --cuda-path= works. I'll disable part of the test that uses sysroot for now and will submit a proper fix once I figure it out. --Artem On Wed, Sep 23, 2015 at 5:07 PM, Rui Ueyama wrote: > I don't know the exact condition, but here's

Re: r248433 - [CUDA] Added CUDA installation detector class.

2015-09-23 Thread Rui Ueyama via cfe-commits
I don't know the exact condition, but here's my results. OK $ bin/clang -v -target=i386-unknown-linux --sysroot=/tmp/x/CUDA clang-3.7: error: unknown argument: '-target=i386-unknown-linux' clang version 3.8.0

Re: r248433 - [CUDA] Added CUDA installation detector class.

2015-09-23 Thread Artem Belevich via cfe-commits
How long is very long? I've tested on ubuntu with ~1600-character-long path and it works. Buildbot filed with a much shorter path: http://lab.llvm.org:8080/green/job/clang-stage1-cmake-RA_check/7416/consoleFull#-1841136826a1ca8a51-895e-46c6-af87-ce24fa4cd561 I'll unroll my change if I fail to fig

Re: r248433 - [CUDA] Added CUDA installation detector class.

2015-09-23 Thread Rui Ueyama via cfe-commits
Yes I can. Create a deep nested directly, create a symlink from there to your Inputs/CUDA, and run "clang -v --target=i386-unknown-linux --sysroot=/Inputs/CUDA". On Wed, Sep 23, 2015 at 4:47 PM, Artem Belevich wrote: > I'm looking into it. Can you reproduce it on linux? > > > On Wed, Sep 23, 201

Re: r248433 - [CUDA] Added CUDA installation detector class.

2015-09-23 Thread Artem Belevich via cfe-commits
I'm looking into it. Can you reproduce it on linux? On Wed, Sep 23, 2015 at 4:43 PM, Rui Ueyama wrote: > On Wed, Sep 23, 2015 at 2:49 PM, Artem Belevich via cfe-commits < > cfe-commits@lists.llvm.org> wrote: > >> Author: tra >> Date: Wed Sep 23 16:49:39 2015 >> New Revision: 248433 >> >> URL: h

Re: r248433 - [CUDA] Added CUDA installation detector class.

2015-09-23 Thread Rui Ueyama via cfe-commits
On Wed, Sep 23, 2015 at 2:49 PM, Artem Belevich via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Author: tra > Date: Wed Sep 23 16:49:39 2015 > New Revision: 248433 > > URL: http://llvm.org/viewvc/llvm-project?rev=248433&view=rev > Log: > [CUDA] Added CUDA installation detector class. > > Ad

r248433 - [CUDA] Added CUDA installation detector class.

2015-09-23 Thread Artem Belevich via cfe-commits
Author: tra Date: Wed Sep 23 16:49:39 2015 New Revision: 248433 URL: http://llvm.org/viewvc/llvm-project?rev=248433&view=rev Log: [CUDA] Added CUDA installation detector class. Added new option --cuda-path= which allows overriding default search paths. If it's not specified we look for CUDA insta