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

[Argyrios Kyrtzidis]

r244187, thanks!

> On Aug 5, 2015, at 6:03 PM, Eric Christopher <echristo at gmail.com> wrote:
> 
> Yep. That works.
> 
> -eric
> 
> On Wed, Aug 5, 2015 at 5:40 PM Argyrios Kyrtzidis <akyrtzi at gmail.com <mailto:akyrtzi at gmail.com>> 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-mangled-name.cpp
> @@ -2,7 +2,7 @@
>  // RUN: c-index-test -write-pch %t_linux.ast -target i686-pc-linux-gnu %s
> 
>  // RUN: c-index-test -test-print-mangle %t_linux.ast | FileCheck %s --check-prefix=ITANIUM
>  
> -// RUN: c-index-test -write-pch %t_macho.ast -arch x86_64 -mmacosx-version-min=10.6 %s
> +// RUN: c-index-test -write-pch %t_macho.ast -target x86_64-apple-darwin %s
> 
>  // RUN: c-index-test -test-print-mangle %t_macho.ast | FileCheck %s --check-prefix=MACHO
>  
>  // RUN: c-index-test -write-pch %t_msft.ast -target i686-pc-win32 %s
> 
> 
>> On Aug 5, 2015, at 5:06 PM, Eric Christopher <echristo at gmail.com <mailto:echristo at gmail.com>> wrote:
>> 
>> 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-test -write-pch /usr/local/google/home/echristo/builds/build-llvm/tools/clang/test/Index/Output/print-mangled-name.cpp.tmp_linux.ast -target i686-pc-linux-gnu /usr/local/google/home/echristo/sources/llvm/tools/clang/test/Index/print-mangled-name.cpp
>> /usr/local/google/home/echristo/builds/build-llvm/Debug+Asserts/bin/c-index-test -test-print-mangle /usr/local/google/home/echristo/builds/build-llvm/tools/clang/test/Index/Output/print-mangled-name.cpp.tmp_linux.ast | /usr/local/google/home/echristo/builds/build-llvm/Debug+Asserts/bin/FileCheck /usr/local/google/home/echristo/sources/llvm/tools/clang/test/Index/print-mangled-name.cpp --check-prefix=ITANIUM
>> /usr/local/google/home/echristo/builds/build-llvm/Debug+Asserts/bin/c-index-test -write-pch /usr/local/google/home/echristo/builds/build-llvm/tools/clang/test/Index/Output/print-mangled-name.cpp.tmp_macho.ast -arch x86_64 -mmacosx-version-min=10.6 /usr/local/google/home/echristo/sources/llvm/tools/clang/test/Index/print-mangled-name.cpp
>> /usr/local/google/home/echristo/builds/build-llvm/Debug+Asserts/bin/c-index-test -test-print-mangle /usr/local/google/home/echristo/builds/build-llvm/tools/clang/test/Index/Output/print-mangled-name.cpp.tmp_macho.ast | /usr/local/google/home/echristo/builds/build-llvm/Debug+Asserts/bin/FileCheck /usr/local/google/home/echristo/sources/llvm/tools/clang/test/Index/print-mangled-name.cpp --check-prefix=MACHO
>> /usr/local/google/home/echristo/builds/build-llvm/Debug+Asserts/bin/c-index-test -write-pch /usr/local/google/home/echristo/builds/build-llvm/tools/clang/test/Index/Output/print-mangled-name.cpp.tmp_msft.ast -target i686-pc-win32 /usr/local/google/home/echristo/sources/llvm/tools/clang/test/Index/print-mangled-name.cpp
>> /usr/local/google/home/echristo/builds/build-llvm/Debug+Asserts/bin/c-index-test -test-print-mangle /usr/local/google/home/echristo/builds/build-llvm/tools/clang/test/Index/Output/print-mangled-name.cpp.tmp_msft.ast | /usr/local/google/home/echristo/builds/build-llvm/Debug+Asserts/bin/FileCheck /usr/local/google/home/echristo/sources/llvm/tools/clang/test/Index/print-mangled-name.cpp --check-prefix=MICROSOFT
>> --
>> Exit Code: 1
>> 
>> Command Output (stderr):
>> --
>> /usr/local/google/home/echristo/sources/llvm/tools/clang/test/Index/print-mangled-name.cpp:13:11: error: expected string not found in input
>> // MACHO: mangled=__Z3fooii
>>           ^
>> <stdin>:1:1: note: scanning from here
>> macro definition=__llvm__ [mangled=]
>> ^
>> <stdin>:1:28: note: possible intended match here
>> macro definition=__llvm__ [mangled=]
>>                            ^
>> 
>> --
>> 
>> -eric
>> 
>> On Wed, Aug 5, 2015 at 4:11 PM Argyrios Kyrtzidis <akyrtzi at gmail.com <mailto:akyrtzi at gmail.com>> 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 <renato.golin at linaro.org <mailto:renato.golin at linaro.org>> wrote:
>> >
>> > On 5 August 2015 at 18:23, Argyrios Kyrtzidis <akyrtzi at gmail.com <mailto:akyrtzi at gmail.com>> 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 <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 broke our bots:
>> >
>> > http://lab.llvm.org:8011/builders/clang-cmake-armv7-a15-full/builds/6914 <http://lab.llvm.org:8011/builders/clang-cmake-armv7-a15-full/builds/6914>
>> >
>> > http://lab.llvm.org:8011/builders/clang-cmake-thumbv7-a15-full-sh/builds/1882 <http://lab.llvm.org:8011/builders/clang-cmake-thumbv7-a15-full-sh/builds/1882>
>> >
>> > and 244076 didn't fix it.
>> >
>> > cheers,
>> > --renato
>> 
>> _______________________________________________
>> cfe-commits mailing list
>> cfe-commits at lists.llvm.org <mailto:cfe-commits at lists.llvm.org>
>> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits <http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits>
> 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150805/ee370567/attachment-0001.html>