Yep. That works. -eric
On Wed, Aug 5, 2015 at 5:40 PM Argyrios Kyrtzidis <akyr...@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 <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> > 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> >> wrote: >> > >> > On 5 August 2015 at 18:23, Argyrios Kyrtzidis <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 >> >> 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-thumbv7-a15-full-sh/builds/1882 >> > >> > and 244076 didn't fix it. >> > >> > cheers, >> > --renato >> >> _______________________________________________ >> cfe-commits mailing list >> cfe-commits@lists.llvm.org >> 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
