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

2016-10-17 Thread 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 uni

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

2016-10-14 Thread Julian Bangert via cfe-commits
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 wrote: > Apologies for the breakage. I investigated and it turns out

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

2016-10-14 Thread Julian Bangert via cfe-commits
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