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
jbangert removed rL LLVM as the repository for this revision.
jbangert updated this revision to Diff 74737.
Herald added subscribers: mgorny, beanz.
https://reviews.llvm.org/D24997
Files:
clang-tidy/utils/ASTUtils.cpp
clang-tidy/utils/ASTUtils.h
clang-tidy/utils/CMakeLists.txt
clang-tidy/
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
jbangert marked an inline comment as done.
jbangert added a comment.
Thanks for the review!
https://reviews.llvm.org/D24997
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jbangert updated this revision to Diff 74311.
https://reviews.llvm.org/D24997
Files:
clang-tidy/utils/ASTUtils.cpp
clang-tidy/utils/ASTUtils.h
clang-tidy/utils/NamespaceAliaser.cpp
clang-tidy/utils/NamespaceAliaser.h
clang-tidy/utils/UsingInserter.cpp
clang-tidy/utils/UsingInserter.h
jbangert updated this revision to Diff 74187.
jbangert marked 15 inline comments as done.
https://reviews.llvm.org/D24997
Files:
clang-tidy/utils/ASTUtils.cpp
clang-tidy/utils/ASTUtils.h
clang-tidy/utils/NamespaceAliaser.cpp
clang-tidy/utils/NamespaceAliaser.h
clang-tidy/utils/UsingInse
jbangert created this revision.
jbangert added a reviewer: alexfh.
jbangert added a subscriber: cfe-commits.
jbangert added a project: clang-tools-extra.
This adds helper classes to add using declaractions and namespace aliases to
function bodies. These help making function calls to deeply nested
x; void b() { x = 1; } };",
+
fieldDecl(hasDefaultMemberInitializer(integerLiteral();
+}
+
TEST(Matcher, NewExpression) {
StatementMatcher New = cxxNewExpr();
On Wed, Mar 2, 2016 at 12:23 PM Richard Smith wrote:
> On Wed, Mar 2, 2016 at 12:06 PM, Aaron Ballman via
sts/ASTMatchers/ASTMatchersTest.cpp
> ===
> --- unittests/ASTMatchers/ASTMatchersTest.cpp (revision 262512)
> +++ unittests/ASTMatchers/ASTMatchersTest.cpp (working copy)
> @@ -2402,6 +2402,13 @@
> hasName("E"
This adds a matcher for C++ in Class initializers.
---
include/clang/ASTMatchers/ASTMatchers.h | 14 ++
unittests/ASTMatchers/ASTMatchersTest.cpp | 7 +++
2 files changed, 21 insertions(+)
diff --git a/include/clang/ASTMatchers/ASTMatchers.h
b/include/clang/ASTMatchers/ASTMatc
10 matches
Mail list logo
