[PATCH] Add .clang-tidy configuration file to provide LLVM-optimized defaults for clang-tidy.

Daniel Jasper djasper at google.com
Mon Sep 8 00:35:04 PDT 2014


I am fine with leaving those out until it causes a problem. On the other
hand, adding them seems cheap. Up to you.


On Mon, Sep 8, 2014 at 9:12 AM, Alexander Kornienko <alexfh at google.com>
wrote:

> The tests are not included in the compilation database, so clang-tidy
> won't run on them. But I could add .clang-tidy files to the test/
> directories, if you like.
> On 8 Sep 2014 09:08, "Chandler Carruth" <chandlerc at gmail.com> wrote:
>
>>
>> On Sun, Sep 7, 2014 at 11:23 PM, Daniel Jasper <djasper at google.com>
>> wrote:
>>
>>> Do we really need a separate file for Clang? Shouldn't D5236 suffice as
>>> Clang is always checked out into LLVM?
>>>
>>
>> There are folks that still develop without Clang checked out under LLVM.
>> Seems cheap to support that here.
>>
>>
>>>
>>> On the other hand, I would add files containing "Checks: '-*'" into the
>>> test/ directories of both. Mosts test won't be (and don't need to be)
>>> clang-tidy clean.
>>>
>>
>> Agreed.
>>
>>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140908/8ef993b7/attachment.html>


More information about the cfe-commits mailing list