Re: r244066 - [test/Index] Update libclang tests to use libclang for creating PCH files.

2015-08-05 Thread Eric Christopher
Yep. That works. -eric On Wed, Aug 5, 2015 at 5:40 PM Argyrios Kyrtzidis wrote: > What about this change: > > diff --git a/test/Index/print-mangled-name.cpp > b/test/Index/print-mangled-name.cpp > index 6dcda70..3d74fe5 100644 > --- a/test/Index/print-mangled-name.cpp > +++ b/test/Index/print-m

Re: r244066 - [test/Index] Update libclang tests to use libclang for creating PCH files.

2015-08-05 Thread Argyrios Kyrtzidis
What about this change: diff --git a/test/Index/print-mangled-name.cpp b/test/Index/print-mangled-name.cpp index 6dcda70..3d74fe5 100644 --- a/test/Index/print-mangled-name.cpp +++ b/test/Index/print-mangled-name.cpp @@ -2,7 +2,7 @@ // RUN: c-index-test -write-pch %t_linux.ast -target i686-pc-li

Re: r244066 - [test/Index] Update libclang tests to use libclang for creating PCH files.

2015-08-05 Thread Eric Christopher
Still broken here locally on my machine after updating: FAIL: Clang :: Index/print-mangled-name.cpp (3844 of 22837) TEST 'Clang :: Index/print-mangled-name.cpp' FAILED Script: -- /usr/local/google/home/echristo/builds/build-llvm/Debug+Asserts/bin/c-index-t

Re: r244066 - [test/Index] Update libclang tests to use libclang for creating PCH files.

2015-08-05 Thread Argyrios Kyrtzidis
r244176. > On Aug 5, 2015, at 4:11 PM, Argyrios Kyrtzidis wrote: > > I tentatively made a change to one of the tests and it looks like it worked, > so I’ll make the same change to the rest. > >> On Aug 5, 2015, at 3:10 PM, Renato Golin wrote: >> >> On 5 August 2015 at 18:23, Argyrios Kyrtzid

Re: r244066 - [test/Index] Update libclang tests to use libclang for creating PCH files.

2015-08-05 Thread Argyrios Kyrtzidis
I tentatively made a change to one of the tests and it looks like it worked, so I’ll make the same change to the rest. > On Aug 5, 2015, at 3:10 PM, Renato Golin wrote: > > On 5 August 2015 at 18:23, Argyrios Kyrtzidis wrote: >> Author: akirtzidis >> Date: Wed Aug 5 12:23:59 2015 >> New Revis

Re: r244066 - [test/Index] Update libclang tests to use libclang for creating PCH files.

2015-08-05 Thread Renato Golin
On 5 August 2015 at 18:23, Argyrios Kyrtzidis wrote: > Author: akirtzidis > Date: Wed Aug 5 12:23:59 2015 > New Revision: 244066 > > URL: http://llvm.org/viewvc/llvm-project?rev=244066&view=rev > Log: > [test/Index] Update libclang tests to use libclang for creating PCH files. Hi, This also bro

r244066 - [test/Index] Update libclang tests to use libclang for creating PCH files.

2015-08-05 Thread Argyrios Kyrtzidis
Author: akirtzidis Date: Wed Aug 5 12:23:59 2015 New Revision: 244066 URL: http://llvm.org/viewvc/llvm-project?rev=244066&view=rev Log: [test/Index] Update libclang tests to use libclang for creating PCH files. This is consistent and tests the primary configuration we want to test, libclang crea