It'd be the issue if the test depended on installed headers. The builder doesn't have MS headers installed.
On Thu, Dec 22, 2016 at 1:27 AM Aaron Ballman <aa...@aaronballman.com> wrote: > On Tue, Dec 20, 2016 at 5:58 PM, Malcolm Parsons > <malcolm.pars...@gmail.com> wrote: > > On 20 December 2016 at 22:32, Aaron Ballman <aa...@aaronballman.com> > wrote: > >> On Tue, Dec 20, 2016 at 4:26 PM, Malcolm Parsons via cfe-commits > >> <cfe-commits@lists.llvm.org> wrote: > >>> Author: malcolm.parsons > >>> Date: Tue Dec 20 15:26:07 2016 > >>> New Revision: 290202 > >>> > >>> URL: http://llvm.org/viewvc/llvm-project?rev=290202&view=rev > >>> Log: > >>> [clang-tidy] Add modernize-use-default-member-init check > >>> > >>> Summary: Fixes PR18858 > >> > >> This appears to have broken one of the bots: > >> > >> http://bb.pgr.jp/builders/ninja-x64-msvc-RA-centos6/builds/33046 > > > > error: unknown type name 'char16_t' > > error: unknown type name 'char32_t' > > > > I see commented tests in other clang-tidy tests: > > > > // TODO: enable these tests once all supported compilers > > // support char16_t and char32_t (VS2013 does not) > > > > // disabled for now until it is clear > > // how to enable them in the test > > //} catch (const char16_t*) { > > We no longer support MSVC 2013, so that should not be an issue. > Takumi, I'm not certain what to make of this builder (MSVC centos?). > What version of Visual Studio is it compiling with? Or is this an > instance where we need to set the -fms-compatibility-version=19 when > running the test with clang-cl? > > ~Aaron > > > > > -- > > Malcolm Parsons >
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
