Re: [clang-tools-extra] r264563 - clang-tidy: Fix broken buildbot

2016-04-05 Thread Richard via cfe-commits
In article , Alexander Kornienko writes: > > On Mon, Mar 28, 2016 at 6:15 AM, Richard Thomson via cfe-commits < > cfe-commits@lists.llvm.org> wrote: > > > Author: legalize > > Date: Sun Mar 27 23:15:41 2016 > > New Revision: 264563 > > > > URL: http://llvm.org/viewvc/llvm-project?rev=264563

Re: [clang-tools-extra] r264563 - clang-tidy: Fix broken buildbot

2016-04-05 Thread Alexander Kornienko via cfe-commits
I'm not sure I understand what this test has to do with VS2013. Clang-tidy should be able to parse this code, and if it doesn't (e.g. due to -fms-compatibility being turned on by default on windows), we should put unsupported constructs under an appropriate #ifdef, not comment them out completely.

[clang-tools-extra] r264563 - clang-tidy: Fix broken buildbot

2016-03-27 Thread Richard Thomson via cfe-commits
Author: legalize Date: Sun Mar 27 23:15:41 2016 New Revision: 264563 URL: http://llvm.org/viewvc/llvm-project?rev=264563&view=rev Log: clang-tidy: Fix broken buildbot VS 2013 does not support char16_t or char32_t Modified: clang-tools-extra/trunk/test/clang-tidy/modernize-raw-string-literal.