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 <echri...@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 <akyr...@gmail.com 
> <mailto:akyr...@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.go...@linaro.org 
> > <mailto:renato.go...@linaro.org>> wrote:
> >
> > On 5 August 2015 at 18:23, Argyrios Kyrtzidis <akyr...@gmail.com 
> > <mailto:akyr...@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@lists.llvm.org <mailto:cfe-commits@lists.llvm.org>
> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits 
> <http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits>

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to