Re: [clang-tools-extra] r286222 - [clang-tidy] clang-analyzer-alpha* checks are not registered, so there's no need to disable them

2016-11-11 Thread Piotr Padlewski via cfe-commits
Hmm, right now there is no way to use alpha checks in clang tidy? Piotr On Nov 10, 2016 10:57, "Malcolm Parsons via cfe-commits" < cfe-commits@lists.llvm.org> wrote: > On 9 November 2016 at 18:50, Devin Coughlin wrote: > > > We agree that this is a valuable checker and are

Re: [clang-tools-extra] r286222 - [clang-tidy] clang-analyzer-alpha* checks are not registered, so there's no need to disable them

2016-11-09 Thread Alexander Kornienko via cfe-commits
Malcolm, does it work for you? On Wed, Nov 9, 2016 at 10:56 AM, Devin Coughlin wrote: > + Anna, Alexander, and Artem. > > > On Nov 9, 2016, at 10:50 AM, Devin Coughlin via cfe-commits < > cfe-commits@lists.llvm.org> wrote: > > > > > >> On Nov 8, 2016, at 9:44 AM, Malcolm

Re: [clang-tools-extra] r286222 - [clang-tidy] clang-analyzer-alpha* checks are not registered, so there's no need to disable them

2016-11-09 Thread Devin Coughlin via cfe-commits
+ Anna, Alexander, and Artem. > On Nov 9, 2016, at 10:50 AM, Devin Coughlin via cfe-commits > wrote: > > >> On Nov 8, 2016, at 9:44 AM, Malcolm Parsons >> wrote: >> >> On 8 November 2016 at 16:59, Alexander Kornienko

Re: [clang-tools-extra] r286222 - [clang-tidy] clang-analyzer-alpha* checks are not registered, so there's no need to disable them

2016-11-09 Thread Devin Coughlin via cfe-commits
> On Nov 8, 2016, at 9:44 AM, Malcolm Parsons wrote: > > On 8 November 2016 at 16:59, Alexander Kornienko wrote: >> On Nov 8, 2016 2:11 AM, "Malcolm Parsons" wrote: >>> Oh, I was using

Re: [clang-tools-extra] r286222 - [clang-tidy] clang-analyzer-alpha* checks are not registered, so there's no need to disable them

2016-11-08 Thread Malcolm Parsons via cfe-commits
On 8 November 2016 at 16:59, Alexander Kornienko wrote: > On Nov 8, 2016 2:11 AM, "Malcolm Parsons" wrote: >> Oh, I was using clang-analyzer-alpha.cplusplus.VirtualCall. >> >> Should clang-tidy have an option to enable experimental checks? > > I'd

Re: [clang-tools-extra] r286222 - [clang-tidy] clang-analyzer-alpha* checks are not registered, so there's no need to disable them

2016-11-08 Thread Alexander Kornienko via cfe-commits
I'd instead ask Static Analyzer folks if they can graduate this check. What is your experience with it? Do you feel it's good enough for mainstream? On Nov 8, 2016 2:11 AM, "Malcolm Parsons" wrote: > On 8 November 2016 at 08:28, Alexander Kornienko via cfe-commits >

Re: [clang-tools-extra] r286222 - [clang-tidy] clang-analyzer-alpha* checks are not registered, so there's no need to disable them

2016-11-08 Thread Malcolm Parsons via cfe-commits
On 8 November 2016 at 08:28, Alexander Kornienko via cfe-commits wrote: > [clang-tidy] clang-analyzer-alpha* checks are not registered, so there's no > need to disable them Oh, I was using clang-analyzer-alpha.cplusplus.VirtualCall. Should clang-tidy have an option

[clang-tools-extra] r286222 - [clang-tidy] clang-analyzer-alpha* checks are not registered, so there's no need to disable them

2016-11-08 Thread Alexander Kornienko via cfe-commits
Author: alexfh Date: Tue Nov 8 02:28:19 2016 New Revision: 286222 URL: http://llvm.org/viewvc/llvm-project?rev=286222=rev Log: [clang-tidy] clang-analyzer-alpha* checks are not registered, so there's no need to disable them Modified: