[PATCH] D24997: [ClangTidy] Add UsingInserter and NamespaceAliaser

[Haojian Wu via cfe-commits]

Yeah, "make -j 32" won't compile binaries in clang-tools-extra repo.

"make check-clang-tools" should work though I don't use make. I usually run
`ninja check-clang-tools` command to build all binaries and run all
lint-tests/unittest in clang-extra-tools. There is a remaining build error
in the unittests, I fixed it and recommitted this patch in r284368.

On Fri, Oct 14, 2016 at 10:26 PM, Julian Bangert <bangert at google.com> wrote:

> I figured out make clang-tidy. Compiles now (the typedef was the wrong way
> around, and i never noticed because make with the default target continued
> to work).
>
>
> Updated the diff.
>
> On Fri, Oct 14, 2016 at 12:49 PM Julian Bangert <bangert at google.com>
> wrote:
>
>> Apologies for the breakage. I investigated and it turns out my
>> open-source checkout does not build clang-tidy. I have checked out llvm
>> into ~/llvm, clang into ~/llvm/tools/clang and clang-extra-tools into
>> ~/llvm/tools/clang/tools/extra.
>> In ~/llvm-build I run cmake ../llvm && make -j 32 and it doesn't compile
>> clang tidy. I also tried  cmake -DCLANG_ENABLE_STATIC_ANALYZER=true
>> ../llvm/ but that doesn't make a difference. Any ideas?
>>
>>
>> On Wed, Oct 12, 2016 at 1:34 AM Haojian Wu <hokein at google.com> wrote:
>>
>> hokein added a comment.
>>
>> @jbangert, your patch broke the buildbot, and I reverted it in r283985.
>> You need to add the new source files to the CMakefile. (I saw other
>> compilation errors after I updated the cmake files, Could you take a look
>> on it? )
>>
>>
>> Repository:
>>   rL LLVM
>>
>> https://reviews.llvm.org/D24997
>>
>>
>>
>>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20161017/56995bfe/attachment.html>